westonpace commented on pull request #11466:
URL: https://github.com/apache/arrow/pull/11466#issuecomment-947100587


   > This only lets us extract from a struct (I'm not so sure it makes sense 
for other types?).
   
   The compute IR currently allows references to be:
     * MapKey (similar to struct, only valid if the key of the map is string I 
think).
     * StructField (covered here)
     * ArraySubscript (I'm pretty sure this means you have a ListArray and you 
want the ith element of each list)
     * ArraySlice (Same as above, operates on list array, seems like overkill 
to me and should probably just be a function call at this point)
   
   Maybe these don't need to be handled in the same way as nested references 
(e.g. maybe we capture an ArraySubscript reference in some other structure) but 
it seems we could cover the MapKey case easily enough at least.


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