Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/11671 )
Change subject: IMPALA-7704: Revert "IMPALA-7644: Hide Parquet page index writing with feature flag" ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/11671/1/tests/query_test/test_parquet_page_index.py File tests/query_test/test_parquet_page_index.py: http://gerrit.cloudera.org:8080/#/c/11671/1/tests/query_test/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/11671/1/tests/query_test/test_parquet_page_index.py@272 PS1, Line 272: e flake8: E501 line too long (92 > 90 characters) http://gerrit.cloudera.org:8080/#/c/11671/1/tests/query_test/test_parquet_page_index.py@354 PS1, Line 354: ) flake8: E501 line too long (91 > 90 characters) -- To view, visit http://gerrit.cloudera.org:8080/11671 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Icf0a64d6ec747275e3ecd6e801e054f81095591a Gerrit-Change-Number: 11671 Gerrit-PatchSet: 1 Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Comment-Date: Fri, 12 Oct 2018 19:17:54 +0000 Gerrit-HasComments: Yes