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.

Sign up for GitHub

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

Jump to bottom

fix Bvt case in issue 14836 #15914

Merged
merged 10 commits into from
May 10, 2024
Merged

fix Bvt case in issue 14836 #15914

merged 10 commits into from
May 10, 2024

Conversation

xzxiong
Copy link
Contributor

@xzxiong xzxiong commented May 8, 2024 •
edited
Loading

What type of PR is this?

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

Which issue(s) this PR fixes:

issue #14836

What this PR does / why we need it:

changes:

  1. move op check statement_info result to the end of bvt.

@mergify mergify bot added the kind/bug Something isn't working label May 8, 2024
@matrix-meow matrix-meow added the size/S Denotes a PR that changes [10,99] lines label May 8, 2024
@mergify mergify bot merged commit a2036c3 into matrixorigin:main May 10, 2024
18 of 19 checks passed
xzxiong added a commit to xzxiong/matrixone that referenced this pull request May 10, 2024
@xzxiong
fix Bvt case in issue 14836 (matrixorigin#15914)
a466d8e 
changes:
1. move op `check statement_info result` to the end of bvt.

Approved by: @heni02
@xzxiong xzxiong mentioned this pull request May 10, 2024
Merged
7 tasks
@xzxiong xzxiong deleted the bvt_14836 branch May 10, 2024 05:14
mergify bot pushed a commit that referenced this pull request May 10, 2024
@xzxiong
fix Bvt case in issue 14836 in 1.2 (#15966)
22d81a3 
ref #15914
changes:
1. move op `check statement_info result` to the end of bvt.

Approved by: @heni02, @sukki37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@heni02 heni02 heni02 approved these changes

@aressu1985 aressu1985 Awaiting requested review from aressu1985 aressu1985 is a code owner

Assignees
No one assigned
Labels
kind/bug Something isn't working size/S Denotes a PR that changes [10,99] lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@xzxiong @heni02 @matrix-meow