Skip to content

Add ignore_order_in for $lookup joined array in compatibility tests - #719

Open
manojnks wants to merge 1 commit into
documentdb:mainfrom
manojnks:lookup-ordering-fix
Open

Add ignore_order_in for $lookup joined array in compatibility tests#719
manojnks wants to merge 1 commit into
documentdb:mainfrom
manojnks:lookup-ordering-fix

Conversation

@manojnks

Copy link
Copy Markdown

Several $lookup compatibility test cases compared the joined array with a strict order, which is not guaranteed across engines. Mark the order-insensitive cases with ignore_order_in=["joined"] and thread test_case.ignore_order_in through to the assertion so these tests pass consistently on MongoDB and DocumentDB.

Several $lookup compatibility test cases compared the joined array with a strict order, which is not guaranteed across engines. Mark the order-insensitive cases with ignore_order_in=["joined"] and thread test_case.ignore_order_in through to the assertion so these tests pass consistently on MongoDB and DocumentDB.

Signed-off-by: Manoj Nandakumar <[email protected]>
@manojnks
manojnks requested a review from a team as a code owner August 17, 2026 18:12
@documentdb-triage-tool documentdb-triage-tool Bot added compatibility test Compatibility test related enhancement New feature or request labels Aug 17, 2026
@documentdb-triage-tool

Copy link
Copy Markdown

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort M · Status Needs Review
Confidence: 0.85 (mixed)

Reasoning

component from path globs (test-coverage); effort from diff stats (12+0 LOC, 4 files); LLM: Fixes non-deterministic test failures in $lookup compatibility tests by handling unguaranteed array ordering across engines, touching test case definitions and assertion logic.

If a label is wrong, remove it manually and ping @patty-chow so the rules can be tuned. The bot will not re-label items that already have component labels.

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

Labels

compatibility test Compatibility test related enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant