[jira] [Commented] (PARQUET-1437) Misleading comment in parquet.thrift

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/PARQUET-1437?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16667015#comment-16667015 ] 

ASF GitHub Bot commented on PARQUET-1437:
-----------------------------------------

zivanfi opened a new pull request #115: PARQUET-1437: Misleading comment in parquet.thrift
URL: https://github.com/apache/parquet-format/pull/115
 
 
   The documentation for list<ColumnOrder> column_orders stated that "Each
   sort order corresponds to one column, determined by its position in the
   list, matching the position of the column in the schema."
   
   However, in reality, while the order of elements in these two
   lists (schema and sort order) are the same, only leaf nodes are
   represented in the list of sort orders, so the positions do not match.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Misleading comment in parquet.thrift
> ------------------------------------
>
>                 Key: PARQUET-1437
>                 URL: https://issues.apache.org/jira/browse/PARQUET-1437
>             Project: Parquet
>          Issue Type: Bug
>          Components: parquet-format
>            Reporter: Zoltan Ivanfi
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: format-2.7.0
>
>
> The documentation for {{list<ColumnOrder> column_orders}} states that "Each sort order corresponds to one column, determined by its position in the list, matching the position of the column in the schema."
> However, in reality, while the order of elements in these two lists (schema and sort order) are the same, only leaf nodes are represented in the list of sort orders, so the positions do *not* match.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)