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

Fix possible logical error "Cannot cancel. Either no query sent or already cancelled" #49106

Merged
merged 3 commits into from May 3, 2023

Conversation

Avogar
Copy link
Member

@Avogar Avogar commented Apr 24, 2023

Changelog category (leave one):

  • Not for changelog (changelog entry is not required)

Closes #49102. Was introduced in #47229

@robot-ch-test-poll robot-ch-test-poll added the pr-not-for-changelog This PR should not be mentioned in the changelog label Apr 24, 2023
@SmitaRKulkarni SmitaRKulkarni self-assigned this Apr 24, 2023
Copy link
Member

@SmitaRKulkarni SmitaRKulkarni left a comment

Choose a reason for hiding this comment

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

LGTM

@tavplubix
Copy link
Member

ClickHouse build check - msan build was broken in master
Compatibility check (amd64) — glibc check failed - was broken in master
Stateless tests (debug) - 01442_merge_detach_attach_long and 00155_long_merges were broken in master
Stateless tests (debug, s3 storage) [6/6] - #49185
Stateless tests (release, analyzer) - check with analyzer is broken in master
Stress test (asan) - #49407
Upgrade check (debug) — OOM in dmesg - we don't have diagnostic tools for OOM in prev release

@robot-ch-test-poll2
Copy link
Contributor

robot-ch-test-poll2 commented May 3, 2023

This is an automated comment for commit 2229f8c with description of existing statuses. It's updated for the latest CI running
The full report is available here
The overall status of the commit is 🟡 pending

Check nameDescriptionStatus
CI runningA meta-check that indicates the running CI. Normally, it's in success or pending state. The failed status indicates some problems with the PR🟡 pending
Mergeable CheckChecks if all other necessary checks are successful🟢 success
Push to DockerhubThe check for building and pushing the CI related docker images to docker hub🟢 success

@tavplubix tavplubix merged commit 01ed70a into master May 3, 2023
11 of 12 checks passed
@tavplubix tavplubix deleted the Avogar-patch-1 branch May 3, 2023 16:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-not-for-changelog This PR should not be mentioned in the changelog
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Logical error: Cannot cancel. Either no query sent or already cancelled
5 participants