Geoffrey Jacoby
Geoffrey Jacoby
@dbwong - we got to ITs this time but the new HA tests had several timeouts (which caused big thread dumps)
@dbwong @stoty - I tried twice to kick this PR's test run off again at https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-PreCommit-GitHub-PR/view/change-requests/job/PR-1430/ but both times it failed in about 38s with an error saying it couldn't...
`[ERROR] org.apache.phoenix.jdbc.FailoverPhoenixConnectionIT.testConnectionWhenActiveZKRestarts Time elapsed: 80.644 s
@bd2019us - changes look good - could you please rebase so that they can be committed?
This is a rebase of #1467 since merging #1465 caused GitHub to incorrectly think there was a merge conflict there. (This is still @jpisaac 's code, I'm just doing it...
This PR has caused InListIT to crash twice in the test environment, and InListIT is a test that it modifies. More investigation is needed before it can be merged. @jpisaac...
@jpisaac - the new test in InListIT was causing OOMs during the iteration where it ran with 50,000 ORs (both in CI and locally). I tweaked it so it runs...
I'll let @jpisaac re-base his original PR now that he's back and close this one.
@ss77892 - are there valid use cases for using an outputcommitter that's not a PathOutputCommitter? If not, is there somewhere we can put some validation in, rather than getting a...