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

delet bvt tag 19912 #20797

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

aressu1985
Copy link
Contributor

@aressu1985 aressu1985 commented Dec 17, 2024

User description

What type of PR is this?

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

Which issue(s) this PR fixes:

issue #19912

delete bvt tag

What this PR does / why we need it:

Removed BVT (Build Verification Test) tag annotations related to issue #19912 from test files


PR Type

Tests


Description

  • Removed BVT (Build Verification Test) tag annotations related to issue [Bug]: r-w occurred during bvt test #19912 from test files
  • Affected test files contain SQL queries to show table structures for system metrics and other system tables
  • No functional changes to the test logic itself

Changes walkthrough 📝

Relevant files
Tests
show_table.sql
Remove BVT tag annotations from show table tests                 

test/distributed/cases/log/show_table.sql

+0/-2     
mo_cloud.sql
Remove BVT tag annotations from mo_cloud tests                     

test/distributed/cases/mo_cloud/mo_cloud.sql

  • Removed BVT tag annotations related to issue [Bug]: r-w occurred during bvt test #19912 from system
    metrics table tests
  • +0/-2     

    💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

    @aressu1985 aressu1985 added this to the 2.1.0 milestone Dec 17, 2024
    @aressu1985 aressu1985 self-assigned this Dec 17, 2024
    @aressu1985 aressu1985 requested a review from heni02 as a code owner December 17, 2024 11:34
    Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ No major issues detected

    @mergify mergify bot added the kind/test-ci label Dec 17, 2024
    Copy link

    Qodo Merge was enabled for this repository. To continue using it, please link your Git account with your Qodo account here.

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    Copy link
    Contributor

    mergify bot commented Dec 19, 2024

    This pull request has been removed from the queue for the following reason: checks failed.

    The merge conditions cannot be satisfied due to failing checks:

    You should look at the reason for the failure and decide if the pull request needs to be fixed or if you want to requeue it.

    If you want to requeue this pull request, you need to post a comment with the text: @mergifyio requeue

    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    kind/test-ci Review effort [1-5]: 1 size/XS Denotes a PR that changes [1, 9] lines
    Projects
    None yet
    Development

    Successfully merging this pull request may close these issues.

    3 participants