Skip to content

[Build/CI] perf-tool container#847

Merged
databend-bot merged 1 commit into
databendlabs:masterfrom
ZhiHanZ:perfs-docker
Jun 17, 2021
Merged

[Build/CI] perf-tool container#847
databend-bot merged 1 commit into
databendlabs:masterfrom
ZhiHanZ:perfs-docker

Conversation

@ZhiHanZ

@ZhiHanZ ZhiHanZ commented Jun 17, 2021

Copy link
Copy Markdown
Contributor

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

Summary

Support to run perf tool in a standalone docker image, easier for CI and testings
Summary about this PR

Changelog

  • Build/Testing/CI

Related Issues

Fixes #issue

Test Plan

Unit Tests

Stateless Tests

@databend-bot databend-bot added the pr-build this PR changes build/testing/ci steps label Jun 17, 2021
@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

Copy link
Copy Markdown
Member

Hello @ZhiHanZ, 🎉 Thank you for opening the pull request! 🎉
Your pull request state is not in Draft, please add Reviewers or Re-request review again!
FuseQuery: @BohuTANG @sundy-li @zhang2014
FuseStore: @drmingdrmer @dantengsky
Or visit datafuse roadmap for some clues.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

Merging #847 (3f3b5b0) into master (064820f) will increase coverage by 0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #847   +/-   ##
======================================
  Coverage      77%     78%           
======================================
  Files         325     326    +1     
  Lines       17834   17858   +24     
======================================
+ Hits        13877   13964   +87     
+ Misses       3957    3894   -63     
Impacted Files Coverage Δ
common/runtime/src/runtime_test.rs 53% <0%> (-21%) ⬇️
common/streams/src/stream_abort.rs 93% <0%> (ø)
fusestore/store/src/meta_service/raftmeta_test.rs 95% <0%> (+<1%) ⬆️
...e/store/src/meta_service/meta_service_impl_test.rs 88% <0%> (+<1%) ⬆️
fusestore/store/src/meta_service/raftmeta.rs 66% <0%> (+<1%) ⬆️
common/planners/src/plan_node.rs 55% <0%> (+<1%) ⬆️
fusequery/query/src/interpreters/plan_scheduler.rs 59% <0%> (+1%) ⬆️
common/planners/src/plan_rewriter.rs 47% <0%> (+2%) ⬆️
...y/src/optimizers/optimizer_projection_push_down.rs 93% <0%> (+2%) ⬆️
...uery/query/src/interpreters/interpreter_setting.rs 95% <0%> (+10%) ⬆️
... and 4 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 064820f...3f3b5b0. Read the comment docs.

@ZhiHanZ
ZhiHanZ requested a review from bohutang June 17, 2021 06:17
@ZhiHanZ ZhiHanZ changed the title [Build/CI] perf-tool containe [Build/CI] perf-tool container Jun 17, 2021

@bohutang bohutang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@databend-bot

Copy link
Copy Markdown
Member

CI Passed
Reviewer Approved
Let's Merge

@databend-bot
databend-bot merged commit d457bed into databendlabs:master Jun 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build this PR changes build/testing/ci steps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants