[GitHub] [phoenix] virajjasani edited a comment on pull request #964: PHOENIX-6222 Bump default HBase version to 2.2.6 and 2.3.3

[GitBox <gi...@apache.org>]

virajjasani edited a comment on pull request #964:
URL: https://github.com/apache/phoenix/pull/964#issuecomment-726728853


   +1 (non-binding) for using latest patch releases.
   However, as for the tests, more often we don't get to see full test runs because tests like `GlobalMutableNonTxIndexWithLazyPostBatchWriteIT`, `IndexUsageIT` crashes VM and then we see multiple skipped tests. For instance, test runs for both builds of this PR so far have more than 100 tests skipped. As of today, do we have any workaround for VM crash or do we know of all tests that usually cause such trouble? Do we generally follow any recommendation like running all tests locally as part of HBase version upgrade to be more confident that all tests are passing (including flaky ones and the ones that are skipped in precommit builds)?
   
   I am just thinking from 5.1 and 4.16 release stability viewpoint that at least if we get to see results of all tests, upgrading HBase versions might not be that much of a big deal and we could see all test results. Thought?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org