tustvold commented on issue #2581:
URL: 
https://github.com/apache/arrow-datafusion/issues/2581#issuecomment-1168882978

   > but I think the datafusion core column needs to be converted to an enum 
consisting of Column and IndexedField
   
   As described above, list index pushdown is likely to yield limited benefits 
for any of the formats we currently support. As such I don't think we need to 
support it in projection pushdown as a first step.
   
   > whereas the relevant data of the sql parsing (the indexed field structure) 
is lost.
   
   I'm not sure I follow what you mean, I would have expected list indexing to 
just be a different type of `PhysicalExpr` without the need to leak into the 
DFSchema at all?


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