You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@paimon.apache.org by "QuakeWang (via GitHub)" <gi...@apache.org> on 2026/06/19 07:07:52 UTC
[PR] [core] Stabilize hybrid RRF tie-breaking [paimon]
QuakeWang opened a new pull request, #8288:
URL: https://github.com/apache/paimon/pull/8288
### Purpose
Hybrid RRF ranked rows within each route only by score. When multiple rows had the same route score, their RRF rank depended on the input bitmap iteration order and Java sort stability, while the final `topK` already used `score desc, rowId asc`.
This PR makes the route-level RRF ranking use the same deterministic tie-break: `score desc, rowId asc`. It also adds regression coverage for tied route scores and verifies the resulting RRF contributions.
### Tests
- `mvn -pl paimon-common -Dtest=HybridSearchRankerTest test`
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [core] Stabilize hybrid RRF tie-breaking [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi commented on PR #8288:
URL: https://github.com/apache/paimon/pull/8288#issuecomment-4749459561
Reviewed the change. The added row-id tie breaker makes the per-route RRF ranking deterministic when route scores are equal, and it is consistent with the existing final top-k tie breaker. The regression test covers the intended case by forcing a non-sorted iteration order.
I also ran the focused test locally:
`mvn -pl paimon-common -Pfast-build -Dtest=HybridSearchRankerTest test`
LGTM from my side.
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] [core] Stabilize hybrid RRF tie-breaking [paimon]
Posted by "JingsongLi (via GitHub)" <gi...@apache.org>.
JingsongLi merged PR #8288:
URL: https://github.com/apache/paimon/pull/8288
--
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: issues-unsubscribe@paimon.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org