westonpace commented on PR #12826:
URL: https://github.com/apache/arrow/pull/12826#issuecomment-1175708617

   I don't think it is an extra project.  The failure is coming from the two 
calls to `Bind` in `ExecNode_Scan`.  The code there is binding both the 
projection and the filter to the dataset schema.  This schema does not include 
the augmented fields and thus the bind fails.  If I remove those two calls to 
bind then everything works.
   
   Inside the scanner, we check both expressions.  If they are not bound we 
bind them to the augmented dataset schema (which isn't exactly available to the 
caller and does include the augmented fields).
   
   So I think the right answer here is to just remove those calls to bind.  
They don't exist on the python equivalent path.  Binding expressions seems like 
an implementation detail anyways and we can probably move away from anyone 
outside arrow-cpp having to know about the concept.


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