BlakeOrth commented on PR #18085:
URL: https://github.com/apache/datafusion/pull/18085#issuecomment-3408539384
This is a much improved way to display the summaries! I was agonizing over
how to use Rust format padding etc. to build out the table, using a
`RecordBatch` to get the formatted output for "free" is clever.
I have a PR queued up that instruments `LIST` that I was going to submit as
soon as
- https://github.com/apache/datafusion/pull/18064
hits main (helps avoid resolving merge conflicts etc). Does it make sense to
hold off finalizing these changes until we get `LIST` instrumented so we can
ensure this behaves as expected with multiple operations?
--
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]