tustvold commented on issue #2564:
URL: https://github.com/apache/arrow-rs/issues/2564#issuecomment-1225878750

   > So based on the above explanation we shouldn't be doing that right since 
the unspecified value can lead to out of bounds access ?
   
   Aah yeah, that's a bug. Fortunately it hasn't been released yet, it was 
changed in https://github.com/apache/arrow-rs/pull/2533. We should fix it 
before the next release.
   
   I do wonder if there is a way to avoid needing to check the null index for 
dictionary arrays, perhaps we could enforce nulls to be 0, and then special 
case an empty dictionary or something... :thinking: 


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

Reply via email to