Skip to content

Add wait expect case.#23739

Closed
Ariznawlll wants to merge 25 commits intomatrixorigin:mainfrom
Ariznawlll:add_wait_expect_case
Closed

Add wait expect case.#23739
Ariznawlll wants to merge 25 commits intomatrixorigin:mainfrom
Ariznawlll:add_wait_expect_case

Conversation

@Ariznawlll
Copy link
Copy Markdown
Contributor

@Ariznawlll Ariznawlll commented Feb 12, 2026

What type of PR is this?

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

Which issue(s) this PR fixes:

replace select sleep(20) to -- @wait_expect(interval, timeout)

issue #
#23738
#9124
#18712

What this PR does / why we need it:

replace select sleep(20) to -- @wait_expect(interval, timeout)

@Ariznawlll Ariznawlll requested a review from heni02 as a code owner February 12, 2026 05:48
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Feb 12, 2026

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ aptend
✅ Ariznawlll
Ariznawl@163.com


Ariznawl@163.com seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@Ariznawlll
Copy link
Copy Markdown
Contributor Author

/auto-test-pr

@Ariznawlll
Copy link
Copy Markdown
Contributor Author

PR #23739 测试覆盖分析

变更摘要

本 PR 主要对 BVT 测试用例进行了维护性修改,将原有的 select sleep(20) 替换为新的 -- @wait_expect(interval, timeout) 标签,以提升测试效率和稳定性。涉及的用例包括 load_datapessimistic_transaction/fulltextpessimistic_transaction/vectorsequencestage 等目录下的 SQL 和 result 文件,以及增加了 Parquet 测试资源文件。

6 类测试覆盖情况

测试类型 覆盖状态 说明
BVT 仅修改和增强 BVT 测试用例,无新增功能代码,所有变更均为 BVT case 和资源文件。
稳定性 未涉及稳定性相关代码或配置,无需关注。
Chaos 未涉及故障注入相关代码或配置,无需关注。
大数据 虽涉及 load_data 相关 case,但本次仅为 BVT 级别 case 维护,无大数据量 case 变更。
PITR 未涉及 PITR 相关 case、配置或代码。
Snapshot 未涉及 Snapshot 相关 case、配置或代码。

图例: ✅ 已覆盖 ⚠️ 需补充 ➖ 不相关

建议

  • 本 PR 已充分覆盖 BVT 测试需求,无需补充其他类型测试。

  • 若后续需扩展大数据或稳定性测试,可基于本 PR 新增的 Parquet 资源,设计大数据量的 load_data 场景。例如:

    大数据 load_data 示例 SQL:

    load data infile {'filepath'='cos://bucket/large/titanic.parquet','format'='parquet'} into table parquet_06;
    select count(*) from parquet_06;
  • 当前 PR 不涉及功能代码、存储、事务、PITR、Snapshot 等模块,无需额外关注其他测试类型。


由 AI Test Analyzer 自动生成 · workflow run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do-not-merge/wip kind/bug Something isn't working size/L Denotes a PR that changes [500,999] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants