Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/15395 )
Change subject: WIP IMPALA-9042: Milestone 1: properly scan files that has full ACID schema ...................................................................... Patch Set 2: (7 comments) http://gerrit.cloudera.org:8080/#/c/15395/2/testdata/bin/generate-schema-statements.py File testdata/bin/generate-schema-statements.py: http://gerrit.cloudera.org:8080/#/c/15395/2/testdata/bin/generate-schema-statements.py@319 PS2, Line 319: ' flake8: E129 visually indented line with same indent as next logical line http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py File tests/query_test/test_scanners.py: http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@210 PS2, Line 210: t flake8: E501 line too long (94 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@1348 PS2, Line 1348: c flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@1352 PS2, Line 1352: c flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@1356 PS2, Line 1356: c flake8: E501 line too long (93 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@1358 PS2, Line 1358: r flake8: E501 line too long (123 > 90 characters) http://gerrit.cloudera.org:8080/#/c/15395/2/tests/query_test/test_scanners.py@1372 PS2, Line 1372: c flake8: E501 line too long (109 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/15395 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic2e2afec00c9a5cf87f1d61b5fe52b0085844bcb Gerrit-Change-Number: 15395 Gerrit-PatchSet: 2 Gerrit-Owner: Zoltan Borok-Nagy <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Thu, 12 Mar 2020 16:45:42 +0000 Gerrit-HasComments: Yes
