Skip to content

Conversation

@LantaoJin
Copy link
Member

@LantaoJin LantaoJin commented Oct 15, 2025

(cherry picked from #4378 commit 3e95147)

The purpose of partially backport #4378 (excluding shard_doc related optimization) is to avoid the plan conflicts between 2.19-dev and main. Since the default order of query results is solid (shardIndex then doc in Lucene, not like the _shard_doc of OpenSearch, no need to generate), no need to add _doc explicitly.
Ref https://github.com/apache/lucene/blob/8e8e37d9e94c290cf8d02e9f318e601baedf28bc/lucene/core/src/java/org/apache/lucene/search/TopDocs.java#L43-L45,

* Partially backport opensearch-project#4378

Signed-off-by: Lantao Jin <ltjin@amazon.com>

---------

Signed-off-by: Lantao Jin <ltjin@amazon.com>
(cherry picked from commit 3e95147)
@LantaoJin LantaoJin changed the title Partially backport #4378 Partially backport #4378 excluding shard_doc Oct 15, 2025
Signed-off-by: Lantao Jin <ltjin@amazon.com>
@LantaoJin LantaoJin marked this pull request as ready for review October 15, 2025 15:10
@LantaoJin LantaoJin changed the title Partially backport #4378 excluding shard_doc [Backport 2.19-dev] Partially backport #4378 excluding shard_doc Oct 15, 2025
@yuancu yuancu merged commit 6b8f420 into opensearch-project:2.19-dev Oct 16, 2025
42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants