Nachiket-Roy commented on PR #19633:
URL: https://github.com/apache/datafusion/pull/19633#issuecomment-3713520586

   how about this? @ethan-tyler I tried tightening the test using 
TestMemoryExec to materialize a {count: UInt64} row as suggested.
   However this causes the test to fail:
   ```bash
   TestMemoryExec::with_new_children not implemented
   ```
   This happens because physical optimizer rules run and require 
ExecutionPlan::with_new_children, which TestMemoryExec does not implement. 
Could you please look into 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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to