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 show subscriptions #13564

Merged
merged 3 commits into from
Dec 19, 2023
Merged

fix show subscriptions #13564

merged 3 commits into from
Dec 19, 2023

Conversation

YANGGMM
Copy link
Contributor

@YANGGMM YANGGMM commented Dec 18, 2023

What type of PR is this?

  • API-change
  • BUG
  • Improvement
  • Documentation
  • Feature
  • Test and CI
  • Code Refactoring

Which issue(s) this PR fixes:

issue #13542

What this PR does / why we need it:

fix show subscriptions

@matrix-meow matrix-meow added the size/S Denotes a PR that changes [10,99] lines label Dec 18, 2023
@mergify mergify bot added the kind/bug Something isn't working label Dec 18, 2023
@daviszhen
Copy link
Contributor

daviszhen commented Dec 18, 2023

@YANGGMM 之前这样修改的原因。是为了解决sql 堆积的问题。也要验证下,如果bvt好加,就加bvt吧。如果bvt不好加,线下验证截图 贴在这里吧。

就是这个bug。#8800

@mergify mergify bot merged commit d265a1d into matrixorigin:main Dec 19, 2023
16 of 18 checks passed
YANGGMM added a commit to YANGGMM/matrixone that referenced this pull request Dec 19, 2023
YANGGMM added a commit to YANGGMM/matrixone that referenced this pull request Dec 19, 2023
YANGGMM added a commit to YANGGMM/matrixone that referenced this pull request Dec 20, 2023
@YANGGMM YANGGMM deleted the fix-13542-1 branch January 22, 2024 12:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Something isn't working size/S Denotes a PR that changes [10,99] lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants