Alex Rodoni has posted comments on this change. ( http://gerrit.cloudera.org:8080/11355 )
Change subject: IMPALA-5937: [DOCS] PARQUET_READ_STATISTICS and PARQUET_DICTIONARY_FILTERING ...................................................................... Patch Set 4: (3 comments) http://gerrit.cloudera.org:8080/#/c/11355/4/docs/topics/impala_parquet_read_statistics.xml File docs/topics/impala_parquet_read_statistics.xml: http://gerrit.cloudera.org:8080/#/c/11355/4/docs/topics/impala_parquet_read_statistics.xml@61 PS4, Line 61: Of the numerical types: bool, integer, floating point > I shared a document with you privately that has should have the details. Pl I added a lame note about the version for the new stats.... http://gerrit.cloudera.org:8080/#/c/11355/5/docs/topics/impala_parquet_read_statistics.xml File docs/topics/impala_parquet_read_statistics.xml: http://gerrit.cloudera.org:8080/#/c/11355/5/docs/topics/impala_parquet_read_statistics.xml@51 PS5, Line 51: blocks > row groups, in this context there is no concept of blocks. Done http://gerrit.cloudera.org:8080/#/c/11355/5/docs/topics/impala_parquet_read_statistics.xml@54 PS5, Line 54: > Maybe we can even merge these two paragraphs since there is some overlap? Removed the second paragraph. -- To view, visit http://gerrit.cloudera.org:8080/11355 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I88fa8c4a64560711251076c50e1695f7f032f9c0 Gerrit-Change-Number: 11355 Gerrit-PatchSet: 4 Gerrit-Owner: Alex Rodoni <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Fredy Wijaya <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Lars Volker <[email protected]> Gerrit-Comment-Date: Mon, 10 Sep 2018 20:51:40 +0000 Gerrit-HasComments: Yes
