Skip to content

Add mysql progress output#1858

Merged
bohutang merged 2 commits into
databendlabs:masterfrom
sundy-li:mysql-stats
Sep 17, 2021
Merged

Add mysql progress output#1858
bohutang merged 2 commits into
databendlabs:masterfrom
sundy-li:mysql-stats

Conversation

@sundy-li

Copy link
Copy Markdown
Member

I hereby agree to the terms of the CLA available at: https://databend.rs/policies/cla/

Summary

Add mysql progress output

mysql> select avg(number) from numbers(100000);
+-------------+
| avg(number) |
+-------------+
|     49999.5 |
+-------------+
1 row in set (0.01 sec)
Read 100000 rows, 800 KB in 0.004 sec., 25 million rows/sec., 200 MB/sec.

Changelog

  • New Feature

Related Issues

Fixes #issue

Test Plan

Unit Tests

Stateless Tests

@databend-bot

Copy link
Copy Markdown
Member

Thanks for the contribution!
I have applied any labels matching special text in your PR Changelog.

Please review the labels and make any necessary changes.

@databend-bot databend-bot added the pr-feature this PR introduces a new feature to the codebase label Sep 16, 2021
@sundy-li
sundy-li marked this pull request as ready for review September 16, 2021 14:05
@codecov-commenter

codecov-commenter commented Sep 16, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1858 (706d0f5) into master (d8bc18d) will increase coverage by 0%.
The diff coverage is 100%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #1858    +/-   ##
=======================================
  Coverage      70%     70%            
=======================================
  Files         635     637     +2     
  Lines       37912   38243   +331     
=======================================
+ Hits        26565   26908   +343     
+ Misses      11347   11335    -12     
Impacted Files Coverage Δ
common/io/src/utils.rs 100% <100%> (ø)
common/io/src/utils_test.rs 100% <100%> (ø)
query/src/sql/plan_parser.rs 73% <0%> (-1%) ⬇️
common/datavalues/src/types/serializations/date.rs 0% <0%> (ø)
...mmon/datavalues/src/types/serializations/number.rs 45% <0%> (ø)
...mmon/datavalues/src/types/serializations/string.rs 20% <0%> (ø)
...mon/datavalues/src/types/serializations/boolean.rs 0% <0%> (ø)
...n/datavalues/src/types/serializations/date_time.rs 0% <0%> (ø)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update d8bc18d...706d0f5. Read the comment docs.

@bohutang

Copy link
Copy Markdown
Member

/lgtm
Great, thank you @sundy-li

@databend-bot

Copy link
Copy Markdown
Member

Approved! Thank you for the PR @sundy-li

@databend-bot

Copy link
Copy Markdown
Member

CI Passed
Reviewer Approved
Let's Merge

1 similar comment
@databend-bot

Copy link
Copy Markdown
Member

CI Passed
Reviewer Approved
Let's Merge

@bohutang
bohutang merged commit b99a6ab into databendlabs:master Sep 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-feature this PR introduces a new feature to the codebase

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants