samuelcolvin opened a new pull request, #12582:
URL: https://github.com/apache/datafusion/pull/12582

   ## Which issue does this PR close?
   
   While working with `MemoryExec`, I noticed it's output is pretty ugly, 
especially with `dbg!` where I'd expect output across multiple lines.
   
   I don't necessarily agree with the subset of fields included, but I didn't 
change it.
   
   ## Rationale for this change
   
   Make `MemoryExec` easy to understand.
   
   ## What changes are included in this PR?
   
   Change the impl of `fmt::Debug` for `MemoryExec` to use the standard 
`debug_struct` approach
   
   ## Are these changes tested?
   
   No, `Debug` implementations generally aren't. I ran it locally.
   
   ## Are there any user-facing changes?
   
   no


-- 
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: github-unsubscr...@datafusion.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to