Igosuki commented on pull request #1006:
URL: https://github.com/apache/arrow-datafusion/pull/1006#issuecomment-939447652


   @houqp Indeed, I mean I could just remove the dict lookup and do it in 
another PR so we could just get array access here. 
   The problem is here 
https://github.com/apache/arrow-datafusion/pull/1006/files#diff-08a2204dc38facfa0757eed25f99d9047fcd3b44bd03df02c89f6f71afacefefR143
 and here 
https://github.com/apache/arrow-datafusion/pull/1006/files#diff-6a157f1b320a619e7a3896027ce7d47a8e3ffeeae8ac3d21fe9157c92ebc2a6bR5218
   If there I have a column, each row may contain a dict, and so if I do a 
reverse lookup, then it should return a value for each non-null row right, but 
instead the dict is treated as a single instance. 


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