simonsssu opened a new issue #2049:
URL: https://github.com/apache/iceberg/issues/2049


   Currently I'm doing some type convert by using Iceberg Accessor. I found 
some tricky codes in Accessor like this:
   
![image](https://user-images.githubusercontent.com/12323514/103965699-48e8e200-5199-11eb-8e22-b1c9e8e5f3cd.png)
   In my understanding, we only need a WrappedPositionAccessor and a 
PositionAccessor, It makes me quite confused why we need a position2accessor 
and position3accessor here . Also if we have more than 3 layers nested types, 
why we can use WrappedPositionAccessor rather than position4accessor ? 
   
   I'm trying to remove position2accessor and position3accessor, just return 
WrappedPositionAccessor for new accessor, and All UTs have passed. I'm not 
understanding the purpose here, if it's a tricky part and we don't have any 
optimization here, I prefer to remove them. 


----------------------------------------------------------------
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:
[email protected]



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

Reply via email to