Jimexist opened a new issue #1169:
URL: https://github.com/apache/arrow-datafusion/issues/1169


   I wonder what you think about moving the creation of the actual 
`RecordBatch` from `ValuesExec::try_new` to `execute` -- the rationale would be 
to make `PhysicalPlan` creation faster and push the actual work into `execute` 
where if can potentially be run concurrently with other parts
   
   Given the size of data in a `VALUES` statement, this is not likely to be any 
real difference so I am fine with leaving the creation in the same place too -- 
I just wanted to mention it.
   
   _Originally posted by @alamb in 
https://github.com/apache/arrow-datafusion/pull/1165#r734957534_


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