samredai opened a new pull request, #4678:
URL: https://github.com/apache/iceberg/pull/4678

   This adds a skeleton visitor for the `BuildPositionAccessors` visitor and 
the following `# TODO` in the struct component of the visitor:
   ```
   # TODO: Populate the `self._index` dictionary where the key is the field ID 
and the value is an accessor for that field.
   #   The equivalent java logic can be found here: 
https://github.com/apache/iceberg/blob/master/api/src/main/java/org/apache/iceberg/Accessors.java#L213-L230
   ```
   The main reason for adding this now is to unblock the work on expressions 
visitors (for example PR #)


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to