rymurr commented on pull request #7290:
URL: https://github.com/apache/arrow/pull/7290#issuecomment-651630439


   I _think_ this is now inline with the spec. The Union/DenseUnion types now 
uses logical type ids in Java. Which is the same as in c++. The difference in a 
java created Union is that the type ids are chosen from a known index (minor 
type). But the java implementation doesn't rely on minor type ordinals strict 
ordering like before. Or at least the intention was to be fully in line with 
spec while maintaining a convenient default choice for type id. Drawing any 
meaning from type buffers alone in client code would be an error in my opinion. 
Hope that clarifies my intention w/ this change.


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to