Skip to content

test: add hive partition fanout BVT coverage - #24522

Merged
mergify[bot] merged 3 commits into
matrixorigin:mainfrom
Ariznawlll:test/hive-partition-fanout-bvt
May 21, 2026
Merged

test: add hive partition fanout BVT coverage#24522
mergify[bot] merged 3 commits into
matrixorigin:mainfrom
Ariznawlll:test/hive-partition-fanout-bvt

Conversation

@Ariznawlll

@Ariznawlll Ariznawlll commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

What type of PR is this?

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

Which issue(s) this PR fixes:

issue #24320

What this PR does / why we need it:

  • add a small hive partition fanout dataset with 10 partitions and 2 parquet files per partition
  • extend hive_partition_external_table BVT to verify multi-file partition scans, partition column fill, __mo_filepath fill, aggregation, and IN pruning

This PR fixes that by:

Verification

  • Built mo-service on origin/main commit 2c49b05 with Go 1.25.10
  • Started a clean local MO service with isolated data-dir /private/tmp/mo-data-fanout-main
  • Ran the added SQL manually and verified:
    • count(*) = 100
    • count(distinct __mo_filepath) = 20
    • each part_id has 10 rows and 2 files
    • part_id in (1,3,9) returns 30 rows

@Ariznawlll
Ariznawlll requested a review from heni02 as a code owner May 21, 2026 07:43
@qodo-code-review

qodo-code-review Bot commented May 21, 2026

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 →

@matrix-meow matrix-meow added the size/S Denotes a PR that changes [10,99] lines label May 21, 2026
@mergify mergify Bot added kind/bug Something isn't working queued labels May 21, 2026
@mergify

mergify Bot commented May 21, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Merge Queue Status

  • ✅ Entered queue — 2026-05-21 11:34 UTC · Rule: main
  • ✅ Checks passed · in-place
  • ✅ Merged — 2026-05-21 13:53 UTC · at 443f1dbbea411dbcf1bc8bfe33eae84ba84f3937 · squash

This pull request spent 2 hours 18 minutes 28 seconds in the queue, including 1 hour 50 seconds running CI.

Required conditions to merge
  • #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • github-review-decision = APPROVED [🛡 GitHub branch protection]
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
    • check-neutral = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
    • check-skipped = Matrixone Compose CI / multi cn e2e bvt test docker compose(PESSIMISTIC)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
    • check-neutral = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
    • check-skipped = Matrixone Standlone CI / Multi-CN e2e BVT Test on Linux/x64(LAUNCH, PROXY)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
    • check-neutral = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
    • check-skipped = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH, PESSIMISTIC)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone CI / SCA Test on Ubuntu/x86
    • check-neutral = Matrixone CI / SCA Test on Ubuntu/x86
    • check-skipped = Matrixone CI / SCA Test on Ubuntu/x86
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone CI / UT Test on Ubuntu/x86
    • check-neutral = Matrixone CI / UT Test on Ubuntu/x86
    • check-skipped = Matrixone CI / UT Test on Ubuntu/x86
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Compose CI / multi cn e2e bvt test docker compose(Optimistic/PUSH)
    • check-neutral = Matrixone Compose CI / multi cn e2e bvt test docker compose(Optimistic/PUSH)
    • check-skipped = Matrixone Compose CI / multi cn e2e bvt test docker compose(Optimistic/PUSH)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH,Optimistic)
    • check-neutral = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH,Optimistic)
    • check-skipped = Matrixone Standlone CI / e2e BVT Test on Linux/x64(LAUNCH,Optimistic)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Upgrade CI / Compatibility Test With Target on Linux/x64(LAUNCH)
    • check-neutral = Matrixone Upgrade CI / Compatibility Test With Target on Linux/x64(LAUNCH)
    • check-skipped = Matrixone Upgrade CI / Compatibility Test With Target on Linux/x64(LAUNCH)
  • any of [🛡 GitHub branch protection]:
    • check-success = Matrixone Utils CI / Coverage
    • check-neutral = Matrixone Utils CI / Coverage
    • check-skipped = Matrixone Utils CI / Coverage

@mergify
mergify Bot merged commit 08661d6 into matrixorigin:main May 21, 2026
23 of 24 checks passed
@mergify mergify Bot removed the queued label May 21, 2026
ULookup pushed a commit to ULookup/matrixone that referenced this pull request May 26, 2026
@Ariznawlll @ULookup
test: add hive partition fanout BVT coverage (matrixorigin#24522)
46f58f6 
- add a small hive partition fanout dataset with 10 partitions and 2 parquet files per partition
- extend hive_partition_external_table BVT to verify multi-file partition scans, partition column fill, __mo_filepath fill, aggregation, and IN pruning

This PR fixes that by:

Approved by: @heni02
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

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

@Ariznawlll @heni02 @matrix-meow