-
Notifications
You must be signed in to change notification settings - Fork 147
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FEAT] Example Analyze for Local Execution Engine #2648
Conversation
samster25
commented
Aug 13, 2024
•
edited
Loading
edited
_ => { | ||
let rt_result = self.runtime_stats.result(); | ||
writeln!(display).unwrap(); | ||
writeln!(display, "rows received = {}", rt_result.rows_received).unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we format the numbers more prettily? maybe using something like https://docs.rs/num-format/latest/num_format/#
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
writeln!(display).unwrap(); | ||
writeln!(display, "rows emitted = {}", rt_result.rows_emitted).unwrap(); | ||
writeln!(display, "bytes read = {}", self.io_stats.load_bytes_read()).unwrap(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also maybe want to pretty format based on the size? i.e. GB if bytes >= 1024^3, MB if bytes >= 1024^2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done