Skip to content

[Build/CI] add build perf-tool to CI#1400

Merged
databend-bot merged 2 commits into
masterfrom
zhihanz/perf-tool-docker
Aug 10, 2021
Merged

[Build/CI] add build perf-tool to CI#1400
databend-bot merged 2 commits into
masterfrom
zhihanz/perf-tool-docker

Conversation

@ZhiHanZ

@ZhiHanZ ZhiHanZ commented Aug 10, 2021

Copy link
Copy Markdown
Contributor

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

Summary

Add performance docker build into CI

Changelog

  • Build/Testing/CI

Related Issues

Test Plan

Unit Tests

Stateless Tests

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

@ZhiHanZ ZhiHanZ changed the title add build perf-tool to CI [Build/CI] add build perf-tool to CI Aug 10, 2021
@ZhiHanZ
ZhiHanZ requested review from PsiACE and bohutang August 10, 2021 06:08
@codecov-commenter

codecov-commenter commented Aug 10, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1400 (8112383) into master (6e58b1c) will increase coverage by 1%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1400     +/-   ##
========================================
+ Coverage      71%     73%     +1%     
========================================
  Files         488     496      +8     
  Lines       28227   31792   +3565     
========================================
+ Hits        20308   23279   +2971     
- Misses       7919    8513    +594     
Impacted Files Coverage Δ
fusestore/store/src/meta_service/raft_types.rs 62% <0%> (-15%) ⬇️
common/datavalues/src/series/de.rs 43% <0%> (-7%) ⬇️
fusestore/store/src/executor/action_handler.rs 57% <0%> (-4%) ⬇️
...ery/src/catalogs/impls/remote_meta_store_client.rs 88% <0%> (-4%) ⬇️
common/datavalues/src/arrays/comparison.rs 35% <0%> (-3%) ⬇️
common/datavalues/src/arrays/ops/scatter.rs 72% <0%> (-2%) ⬇️
common/datavalues/src/arrays/ops/group_hash.rs 51% <0%> (-2%) ⬇️
common/datavalues/src/series/wrap.rs 55% <0%> (-2%) ⬇️
fusestore/store/src/meta_service/state_machine.rs 75% <0%> (-1%) ⬇️
common/datavalues/src/data_field.rs 76% <0%> (-1%) ⬇️
... and 65 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 6e58b1c...8112383. Read the comment docs.

name: Push to Dockerhub
on:
push:
branches: master

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Only consider building when the *.rs/*.toml file is changed?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

updated path dependencies

@ZhiHanZ
ZhiHanZ requested a review from PsiACE August 10, 2021 07:07
@databend-bot

Copy link
Copy Markdown
Member

CI Passed
Reviewer Approved
Let's Merge

@databend-bot
databend-bot merged commit b057e92 into master Aug 10, 2021
@databend-bot
databend-bot deleted the zhihanz/perf-tool-docker branch August 10, 2021 07:49
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