search

datafuselabs / databend

𝗗𝗮𝘁𝗮, 𝗔𝗻𝗮𝗹𝘆𝘁𝗶𝗰𝘀 & 𝗔𝗜. Modern alternative to Snowflake. Cost-effective and simple for massive-scale analytics. https://databend.com
https://docs.databend.com
Other
7.31k stars 704 forks source link

ISSUE-1787: add running difference function #1961

Closed ygf11 closed 2 years ago

ygf11 commented 2 years ago

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

Summary

Add runningDifference function.

Changelog

Related Issues

Fixes #1787

Test Plan

Unit Tests

Stateless Tests

databend-bot commented 2 years ago

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.

codecov-commenter commented 2 years ago

Codecov Report

Merging #1961 (594f2f7) into master (c6823b9) will increase coverage by 0%. The diff coverage is 89%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master   #1961    +/-   ##
=======================================
  Coverage      68%     69%            
=======================================
  Files         639     652    +13     
  Lines       36180   36674   +494     
=======================================
+ Hits        24913   25311   +398     
- Misses      11267   11363    +96
Impacted Files Coverage Δ
common/functions/src/scalars/function_factory.rs 100% <ø> (ø)
.../src/scalars/others/running_difference_function.rs 68% <68%> (ø)
common/functions/src/scalars/others/other.rs 85% <85%> (ø)
...scalars/others/running_difference_function_test.rs 99% <99%> (ø)
query/src/configs/config.rs 50% <0%> (-30%) :arrow_down:
common/datavalues/src/arrays/ops/to_values.rs 9% <0%> (-14%) :arrow_down:
query/src/datasources/table/csv/csv_table.rs 64% <0%> (-12%) :arrow_down:
common/datavalues/src/arrays/list/mod.rs 47% <0%> (-8%) :arrow_down:
common/base/src/uniq_id.rs 93% <0%> (-7%) :arrow_down:
query/src/datasources/table/null/null_table.rs 81% <0%> (-7%) :arrow_down:
... and 154 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 c6823b9...594f2f7. Read the comment docs.

BohuTANG commented 2 years ago

/review @sundy-li

databend-bot commented 2 years ago

Take the reviewer to sundy-li

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge

databend-bot commented 2 years ago

CI Passed Reviewer Approved Let's Merge