[ https://issues.apache.org/jira/browse/IMPALA-7644?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16635790#comment-16635790 ]
Tim Armstrong commented on IMPALA-7644: --------------------------------------- It sounds like we're in agreement, but just wanted to say that I think the carrying cost of a flag for this isn't too bad, since it's just turning off a code path rather than adding an alternative code path. This seems like a reasonable way to hide code for production envs. > Hide Parquet page index writing with feature flag > ------------------------------------------------- > > Key: IMPALA-7644 > URL: https://issues.apache.org/jira/browse/IMPALA-7644 > Project: IMPALA > Issue Type: Improvement > Components: Backend > Affects Versions: Impala 3.1.0 > Reporter: Zoltán Borók-Nagy > Assignee: Zoltán Borók-Nagy > Priority: Major > Labels: parquet, performance > > Currently there is no released Impala version that can write the Parquet page > index: > [https://github.com/apache/parquet-format/blob/master/PageIndex.md] > However, the current Impala master writes the page index since IMPALA-5842, > but cannot read it. > I think we should hide the write path with a feature flag until Impala is > able to read it back and has better test coverage on it. -- This message was sent by Atlassian JIRA (v7.6.3#76005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-all-unsubscr...@impala.apache.org For additional commands, e-mail: issues-all-h...@impala.apache.org