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

Do not advertise @span decorator #8120

Merged
merged 1 commit into from Aug 23, 2023

Conversation

crusaderky
Copy link
Collaborator

@crusaderky crusaderky self-assigned this Aug 21, 2023
@github-actions
Copy link
Contributor

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

       21 files  ±  0         21 suites  ±0   11h 19m 40s ⏱️ - 31m 1s
  3 773 tests ±  0    3 661 ✔️ ±  0     108 💤 ±0  4 ±0 
36 483 runs  +11  34 668 ✔️ +17  1 811 💤  - 6  4 ±0 

For more details on these failures, see this check.

Results for commit 9e9cda2. ± Comparison against base commit 716d526.

@hendrikmakait hendrikmakait self-requested a review August 23, 2023 10:47
Copy link
Member

@hendrikmakait hendrikmakait left a comment

Choose a reason for hiding this comment

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

Thanks, @crusaderky

@hendrikmakait hendrikmakait merged commit 349d1ef into dask:main Aug 23, 2023
24 of 29 checks passed
@crusaderky crusaderky deleted the no_span_decorator branch August 24, 2023 13:42
crusaderky added a commit to crusaderky/dask that referenced this pull request Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can we demistify the order of @annotate and @delayed?
2 participants