Skip to content

[Build/CI] cross compile for arm-based architecture#1689

Merged
databend-bot merged 2 commits into
masterfrom
cross-compile
Sep 1, 2021
Merged

[Build/CI] cross compile for arm-based architecture#1689
databend-bot merged 2 commits into
masterfrom
cross-compile

Conversation

@ZhiHanZ

@ZhiHanZ ZhiHanZ commented Sep 1, 2021

Copy link
Copy Markdown
Contributor

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

Summary

resolve cross compile dependency issues during release
https://github.com/ZhiHanZ/datafuse/runs/3481834157?check_suite_focus=true

Changelog

  • Build/Testing/CI

Related Issues

fixes #1688

Test Plan

Unit Tests

Stateless Tests

@databend-bot databend-bot added the pr-build this PR changes build/testing/ci steps label Sep 1, 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 requested a review from bohutang September 1, 2021 08:50

@github-actions github-actions Bot left a comment

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.

license-eye has totally checked 1556 files.

Valid Invalid Ignored Fixed
762 1 793 0
Click to see the invalid file list
  • Cross.toml

Comment thread Cross.toml
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

@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
Thank you @ZhiHanZ

@codecov-commenter

codecov-commenter commented Sep 1, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1689 (fccd0ec) into master (f0fa13a) will increase coverage by 0%.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master   #1689   +/-   ##
======================================
  Coverage      72%     72%           
======================================
  Files         576     576           
  Lines       33880   33880           
======================================
+ Hits        24464   24472    +8     
+ Misses       9416    9408    -8     
Impacted Files Coverage Δ
store/src/meta_service/raftmeta.rs 93% <0%> (+1%) ⬆️
...pelines/transforms/transform_aggregator_partial.rs 90% <0%> (+1%) ⬆️

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 f0fa13a...fccd0ec. Read the comment docs.

@databend-bot

Copy link
Copy Markdown
Member

CI Passed
Reviewer Approved
Let's Merge

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.

[Build/CI] use customized build tool for cross compile

4 participants