Skip to content

cherry-pick: add MongoDB external table scan - #26494

Closed
iamlinjunhong wants to merge 1 commit into
matrixorigin:4.2-dev-oldfrom
iamlinjunhong:d-26229
Closed

cherry-pick: add MongoDB external table scan#26494
iamlinjunhong wants to merge 1 commit into
matrixorigin:4.2-dev-oldfrom
iamlinjunhong:d-26229

Conversation

@iamlinjunhong

Copy link
Copy Markdown
Contributor

Add tenant-scoped MongoDB connection DDL and catalog metadata, secure driver-backed scanning, conversion and predicate semantics, downstream aggregation support, and complete ALTER/DROP/account lifecycle handling across CNs. Include one connector E2E CI job, regression coverage, and customer-neutral operational templates.

What type of PR is this?

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

Which issue(s) this PR fixes:

issue #26229

What this PR does / why we need it:

feat: add MongoDB external table scan

@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 →

@matrix-meow matrix-meow added the size/XXL Denotes a PR that changes 2000+ lines label Jul 31, 2026
@iamlinjunhong iamlinjunhong changed the title feat: add MongoDB external table scan cherry-pick: add MongoDB external table scan Jul 31, 2026

@XuPeng-SH XuPeng-SH left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed exact head e71d3de.

No blocking issues found. I audited the 4.2 backport delta against the already-reviewed main implementation, including tenant/catalog generation validation, credential-free plan/RPC serialization, cursor/lease/limiter cleanup, connection retirement races and tombstone pruning, operator Reset/Free/remote round-trip, DDL/account/upgrade lifecycle, conservative predicate pushdown with MO residuals, and BSON conversion/nullability boundaries.

Local verification passed: exact-head make build; full mongodb and mongoscan tests (also under -race); Mongo-focused planner, compile, frontend, CN, bootstrap, parser and config tests; and the backported fill/timewin/aggexec package tests. The PR's MongoDB and general CI checks are green at this head.

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

Labels

size/XXL Denotes a PR that changes 2000+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants