You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2021/10/19 21:00:10 UTC

[GitHub] [arrow] westonpace commented on pull request #11466: ARROW-13987: [C++] Support nested field refs

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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org