zanmato1984 commented on issue #49214:
URL: https://github.com/apache/arrow/issues/49214#issuecomment-3882791521

   Hi @owenowenisme , I think this is a good finding and quite reasonable 
suggestions. However I do want to point out that by using the expression 
evaluation (via `ExecuteScalarExpression` - bind once, run acorss batches), the 
`MemoTable`, built at the bind time, seems to get reused across batches. Maybe 
you can see if this is an appropriate for your use case?
   
   That said, I do also think enabling `MemoTable` reusing is still valid for 
low-level function invocations (via `CallFunction`). I would also ask @pitrou 
for opinions.
   
   Thanks for this!


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