Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Upgrade minitrace related crates to 0.6.5 #15530

Merged
merged 2 commits into from
May 15, 2024

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented May 15, 2024

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

Updates the minitrace and minitrace-opentelemetry crate versions to 0.6.5 and removes the patch section for these crates in Cargo.toml.

  • Updates the minitrace crate version to 0.6.5 and includes the enable feature.
  • Updates the minitrace-opentelemetry crate version to 0.6.5.
  • Removes the previously specified git patch for both minitrace and minitrace-opentelemetry from the [patch.crates-io] section, aligning with the task to use the crates.io versions directly.

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test - Explain why

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe): deps

For more details, open the Copilot Workspace session.


This change is Reviewable

Copy link
Contributor

Pull request description must contain CLA like the following:

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

## Summary

Summary about this PR

- Close #issue

1 similar comment
Copy link
Contributor

Pull request description must contain CLA like the following:

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

## Summary

Summary about this PR

- Close #issue

Copy link
Contributor

At least one test kind must be checked in the PR description.
@Xuanwo please update it 🙏.

@github-actions github-actions bot added the pr-chore this PR only has small changes that no need to record, like coding styles. label May 15, 2024
Copy link
Contributor

At least one test kind must be checked in the PR description.
@Xuanwo please update it 🙏.

Copy link
Contributor

At least one type of change must be checked in the PR description.
@Xuanwo please update it 🙏.

1 similar comment
Copy link
Contributor

At least one type of change must be checked in the PR description.
@Xuanwo please update it 🙏.

@Xuanwo Xuanwo requested review from BohuTANG and andylokandy May 15, 2024 03:52
@Xuanwo Xuanwo enabled auto-merge May 15, 2024 03:57
@Xuanwo Xuanwo added this pull request to the merge queue May 15, 2024
@BohuTANG BohuTANG removed this pull request from the merge queue due to a manual request May 15, 2024
@BohuTANG BohuTANG merged commit 6f5e751 into main May 15, 2024
71 checks passed
@BohuTANG BohuTANG deleted the Xuanwo/upgrade-minitrace branch May 15, 2024 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-chore this PR only has small changes that no need to record, like coding styles.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants