Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11563 )
Change subject: IMPALA-7644: Hide Parquet page index writing with feature flag ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/11563/1/tests/custom_cluster/test_parquet_page_index.py File tests/custom_cluster/test_parquet_page_index.py: http://gerrit.cloudera.org:8080/#/c/11563/1/tests/custom_cluster/test_parquet_page_index.py@173 PS1, Line 173: l flake8: E129 visually indented line with same indent as next logical line http://gerrit.cloudera.org:8080/#/c/11563/1/tests/custom_cluster/test_parquet_page_index.py@272 PS1, Line 272: e flake8: E501 line too long (92 > 90 characters) http://gerrit.cloudera.org:8080/#/c/11563/1/tests/custom_cluster/test_parquet_page_index.py@346 PS1, Line 346: ) flake8: E501 line too long (91 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/11563 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If9994882aa59cbaf3ae464100caa8211598287bc Gerrit-Change-Number: 11563 Gerrit-PatchSet: 1 Gerrit-Owner: Zoltan Borok-Nagy <borokna...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Tue, 02 Oct 2018 12:26:32 +0000 Gerrit-HasComments: Yes