Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/13993 )
Change subject: IMPALA-8834: Short-circuit partition key scan ...................................................................... Patch Set 11: (5 comments) http://gerrit.cloudera.org:8080/#/c/13993/11/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java File fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java: http://gerrit.cloudera.org:8080/#/c/13993/11/fe/src/main/java/org/apache/impala/planner/HdfsScanNode.java@211 PS11, Line 211: // The MultiAggregateInfo from the query block of this scan node. Used for determining if line too long (91 > 90) http://gerrit.cloudera.org:8080/#/c/13993/11/tests/query_test/test_queries.py File tests/query_test/test_queries.py: http://gerrit.cloudera.org:8080/#/c/13993/11/tests/query_test/test_queries.py@246 PS11, Line 246: class TestPartitionKeyScans(ImpalaTestSuite): flake8: E302 expected 2 blank lines, found 1 http://gerrit.cloudera.org:8080/#/c/13993/11/tests/query_test/test_queries.py@254 PS11, Line 254: \ flake8: E502 the backslash is redundant between brackets http://gerrit.cloudera.org:8080/#/c/13993/11/tests/query_test/test_queries.py@257 PS11, Line 257: flake8: E203 whitespace before ':' http://gerrit.cloudera.org:8080/#/c/13993/11/tests/query_test/test_queries.py@257 PS11, Line 257: : flake8: E231 missing whitespace after ':' -- To view, visit http://gerrit.cloudera.org:8080/13993 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I26c87525a4f75ffeb654267b89948653b2e1ff8c Gerrit-Change-Number: 13993 Gerrit-PatchSet: 11 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Tue, 03 Sep 2019 06:27:33 +0000 Gerrit-HasComments: Yes
