Sahil Takiar has posted comments on this change. ( http://gerrit.cloudera.org:8080/14248 )
Change subject: IMPALA-8944: Update and re-enable S3PlannerTest ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/14248/1/fe/src/test/java/org/apache/impala/planner/PlannerTest.java File fe/src/test/java/org/apache/impala/planner/PlannerTest.java: http://gerrit.cloudera.org:8080/#/c/14248/1/fe/src/test/java/org/apache/impala/planner/PlannerTest.java@57 PS1, Line 57: HBaseTestDataRegionAssignment assignment = new HBaseTestDataRegionAssignment(); > nit: Add a quick commit why this is needed? Kinda not obvious. Done -- To view, visit http://gerrit.cloudera.org:8080/14248 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1690b6c5346376c1111fd4845c72062cc237e0f9 Gerrit-Change-Number: 14248 Gerrit-PatchSet: 1 Gerrit-Owner: Sahil Takiar <stak...@cloudera.com> Gerrit-Reviewer: Bharath Vissapragada <bhara...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Sahil Takiar <stak...@cloudera.com> Gerrit-Comment-Date: Wed, 18 Sep 2019 19:58:38 +0000 Gerrit-HasComments: Yes