krishan1390 opened a new pull request, #18484: URL: https://github.com/apache/pinot/pull/18484
## Summary - `DuoSegmentDataManager` wraps two `IndexSegment`s under one segment name (e.g. the committed immutable and the still-mutable consuming segment during a realtime commit window). `SegmentDataManager#getSegment()` returns only the primary, so the secondary's rows were silently missed by the query path during the duo window. - `SingleTableExecutionInfo#create` now uses `hasMultiSegments()`/`getSegments()` to expand multi-segment managers into the index segment list for both upsert and non-upsert tables. The expansion is extracted into a small private helper so the two branches stay in sync. ## Test plan - [x] New `SingleTableExecutionInfoTest` covers: duo segment expansion in the non-upsert path, duo with a released secondary, plain single-segment managers preserve previous behavior, and the `TableNotFoundException` path. - [x] `./mvnw -pl pinot-core -am -Dtest=SingleTableExecutionInfoTest test` — 4 tests pass. - [x] `./mvnw spotless:apply checkstyle:check license:check -pl pinot-core` — clean. 🤖 Generated with [Claude Code](https://claude.com/claude-code) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
