Repository navigation
Conversation
xiangfu0
force-pushed
the
xiangfu0/codex1/sorted-exchange-core
branch
from
September 30, 2026 15:54
6843676 to
555f55c
Compare
xiangfu0
force-pushed
the
xiangfu0/codex1/sorted-exchange-ordering
branch
from
September 30, 2026 15:54
5dc4fb3 to
697a710
Compare
xiangfu0
marked this pull request as ready for review
September 30, 2026 16:00
xiangfu0
force-pushed
the
xiangfu0/codex1/sorted-exchange-core
branch
from
September 30, 2026 19:06
555f55c to
6dff651
Compare
xiangfu0
force-pushed
the
xiangfu0/codex1/sorted-exchange-ordering
branch
2 times, most recently
from
October 3, 2026 19:45
986291e to
8e86cc7
Compare
Set streamingSortedMailboxReceive=true to merge leaf ORDER BY on one physical table and stream LIMIT/OFFSET. Match producer collations and reject sender disorder across blocks.
Sample resource usage and the active deadline periodically, including single-sender blocks that pass through without a size cap.
Keep physical leaf and matching-collation coverage while reusing existing offset and limit checks.
Keep the mailbox poll assertion compiling with the simplified parent receiver. Ordering and cancellation behavior remain the same.
xiangfu0
force-pushed
the
xiangfu0/codex1/sorted-exchange-ordering
branch
from
October 3, 2026 20:25
8e86cc7 to
755721f
Compare
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## xiangfu0/codex1/sorted-exchange-core #19711 +/- ##
==========================================================================
- Coverage 68.25% 68.19% -0.06%
Complexity 1450 1450
==========================================================================
Files 3525 3526 +1
Lines 229231 229252 +21
Branches 36381 36389 +8
==========================================================================
- Hits 156458 156342 -116
- Misses 60502 60611 +109
- Partials 12271 12299 +28
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
Contributor
Author
|
CI follow-up at All 14 current-head checks passed: unit/integration, compatibility, quickstart, Java 11 client, linter and Trivy (checks). Maintainer review remains pending. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Streams eligible leaf ORDER BY through a logical k-way merge exchange. Matching typed or single-physical-table leaf selections retain sender ordering; the distinct merge receive applies finite global LIMIT/OFFSET after merging. Hybrid/logical expansion retains the MSE Sort. Plain sends carry ordered input without mailbox markers, structural sender checks or production row validation.
The option remains off by default. The inherited broker gate enables new merge nodes only for homogeneous matching immutable release versions; missing, unknown, mixed, unreadable or SNAPSHOT versions and multi-cluster queries keep the compatible receiver-sort plan. This cached gate is a point-in-time check. Global window receiver sorts preserve complete input beyond response caps; the agreed stalled-sender memory followup remains #19395.
Final stacked head:
755721f7fd1548f3e94f81c86f4f64cfae4c66c8, on corrected coreb047dd479ea360f19cc8ea13595a874590ed66de. Its native diff contains 24 ordering feature files. All original feature commits remain in order with their author/full-message/author-date preserved; source matches the validated input tree exactly.Validation: 526 affected JDK 25 warning/deprecation-enabled cases, plus 376 overlapping full planner/rule/window cases; zero failures/errors/skips or added-line warnings. Applicable formatting, license and checkstyle checks pass. Hosted CI for the final remote head and reviewer/base decisions remain open; local results do not represent remote green checks.
The inherited fixture followup independently passes all 593 ResourceBasedQueryPlansTest cases on this exact source tree with compiler warnings/deprecations enabled, zero failures/errors/skips and all four planner formatting/license/style goals passing. Its native feature patch is byte-identical to the preceding published stack; only core owns the 124 corrected window-input fetch expectations. These local results do not establish hosted CI status.
The existing stack base is retained. Full unit/integration PR workflows are filtered to master; no complete hosted CI result exists for this child at its current stack base. After the prerequisite PRs land, the agreed base update and exact-head CI are still required. No merge, automatic merge, CI suppression or master integration was performed.