Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/12413 )
Change subject: IMPALA-8163: Add banner showing for local catalog mode when enabled. ...................................................................... Patch Set 2: (3 comments) http://gerrit.cloudera.org:8080/#/c/12413/2/be/src/util/default-path-handlers.cc File be/src/util/default-path-handlers.cc: http://gerrit.cloudera.org:8080/#/c/12413/2/be/src/util/default-path-handlers.cc@290 PS2, Line 290: document->GetAllocator()); tab used for whitespace http://gerrit.cloudera.org:8080/#/c/12413/2/tests/custom_cluster/test_local_catalog.py File tests/custom_cluster/test_local_catalog.py: http://gerrit.cloudera.org:8080/#/c/12413/2/tests/custom_cluster/test_local_catalog.py@316 PS2, Line 316: class TestObservability(CustomClusterTestSuite): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/12413/2/tests/custom_cluster/test_local_catalog.py@396 PS2, Line 396: flake8: W391 blank line at end of file -- To view, visit http://gerrit.cloudera.org:8080/12413 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Iaeb2c7b20742a2630d9509da9b629d5f45c55207 Gerrit-Change-Number: 12413 Gerrit-PatchSet: 2 Gerrit-Owner: Anurag Mantripragada <[email protected]> Gerrit-Reviewer: Anurag Mantripragada <[email protected]> Gerrit-Reviewer: Bharath Vissapragada <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Sat, 09 Feb 2019 00:32:09 +0000 Gerrit-HasComments: Yes
