ChinmaySKulkarni commented on a change in pull request #923: URL: https://github.com/apache/phoenix/pull/923#discussion_r505888924
########## File path: phoenix-core/src/main/java/org/apache/phoenix/query/ConnectionQueryServicesImpl.java ########## @@ -4006,6 +4037,33 @@ private PhoenixConnection upgradeSystemTask(PhoenixConnection metaConnection) "ALTER TABLE " + taskTableFullName + " SET " + TTL + "=" + TASK_TABLE_TTL); clearCache(); } + // If SYSTEM.TASK does not have disabled regions split policy, + // set it up here while upgrading it + try (Admin admin = metaConnection.getQueryServices().getAdmin()) { Review comment: Is it possible to add a test case inside BackwardsCompatibilityIT to make sure that the upgrade path changes work and SYSTEM.TASK comes up with the expected split policy? ---------------------------------------------------------------- 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: us...@infra.apache.org