nuno-faria opened a new issue, #24203:
URL: https://github.com/apache/datafusion/issues/24203

   ### Describe the bug
   
   Some of the metrics in the explain plans have inconsistent units. For 
example:
   
   ```
   output_bytes=7.5 GB
   bytes_scanned=1.26 B
   ```
   
   Here `bytes_scanned` is actually 1.26GB, but the formatting is misleading. 
The problem is that this is formatted with `human_readable_count`, while the 
`output_bytes` is formatted with `human_readable_size` (from 
datafusion/common/src/display/human_readable.rs).
   
   Another issue is that size units are suffixed with K,M,**G**,T, while other 
units are K,M,**B**,T.
   
   
   ### To Reproduce
   
   Run `EXPLAIN ANALYZE`.
   
   ### Expected behavior
   
    I think it would be better to format all units with K,M,G,T, and remove the 
B suffix from the size metrics, since the names already contain the word 
"bytes". So the example above would be:
   
   ```
   output_bytes=7.5 G
   bytes_scanned=1.26 G
   ```
   
   cc: @2010YOUY01 what do you think?
   
   ### Additional context
   
   _No response_


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