Skip to content

remove ut - #26764

Open
daviszhen wants to merge 4 commits into
matrixorigin:mainfrom
daviszhen:0806-remove-time-heavy-ut
Open

remove ut#26764
daviszhen wants to merge 4 commits into
matrixorigin:mainfrom
daviszhen:0806-remove-time-heavy-ut

Conversation

@daviszhen

Copy link
Copy Markdown
Contributor

What type of PR is this?

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

Which issue(s) this PR fixes:

issue #26699

What this PR does / why we need it:

TestPartitionBasedShardCanBeCreated 耗时119s。

默认启动的 MatrixOne 不会实际启动 shard service。先注释掉此ut

它原本验证的场景是:

  1. 通过 runShardClusterTest 使用 shard 专用的共享 3CN 测试集群;
  2. 在 CN0 创建独立数据库;
  3. 创建一个包含 2 个分区的分区表;
  4. 从 mo_catalog.mo_tables 获取父表 ID;
  5. 通过 CN0 的 ShardStorage.Get 读取分片元数据;
  6. 断言:
    • 返回的父表 ID 正确;
    • Policy == Partition;
    • ShardsCount == 2;
    • 有两个 partition shard ID;
    • 用每个 partition ID 查询时,都能映射回同一父表和同一份分片元数据。

它验证的是“创建分区表时,shard metadata 是否自动生成且父表/分区 ID 映射正确”,不覆盖插入、路由、读写或副本均衡。

@daviszhen
daviszhen requested a review from XuPeng-SH as a code owner August 6, 2026 12:52
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

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

Labels

kind/enhancement size/S Denotes a PR that changes [10,99] lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants