houqp edited a comment on pull request #1006:
URL: https://github.com/apache/arrow-datafusion/pull/1006#issuecomment-939563050


   I think there might be some confusions here. In my mind, `col["a"]["b"]` 
should access `col` as a dictionary in the sense of hashmap, not Arrow 
dictionary encoded array. In other words, when index key is a string, I think 
we should be matching for Struct datatype instead of dictionary type.
   
   BTW, I am perfectly fine with us splitting the PRs into two and handle the 
dictionary access as a follow up if you prefer to do so :+1: 


-- 
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-unsubscr...@arrow.apache.org

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


Reply via email to