[GitHub] [incubator-iceberg] rdblue commented on issue #598: Support recognizing the column represented with an array of fields

[GitBox <gi...@apache.org>]

rdblue commented on issue #598: Support recognizing the column represented with an array of fields
URL: https://github.com/apache/incubator-iceberg/issues/598#issuecomment-548497847
 
 
   Right now, we use a column key to do this to avoid traversing the schema. The field name array should be converted to dot notation: `["a", "b.c"]` gets converted to `"a.b.c"` and that key is looked up in the field index.
   
   I don't think we need to add an array version. There isn't much value and that makes the API more confusing. For example, users may think it is better to use the array version and attempt to parse identifiers, when there is no need to do it. The point of this simplification is to avoid needing to parse identifiers.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to 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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@iceberg.apache.org
For additional commands, e-mail: issues-help@iceberg.apache.org