Skip to content

Conversation

@ajleong623
Copy link
Contributor

@ajleong623 ajleong623 commented Jun 20, 2025

Description

Solves the wildcard sorting issue.

Issues Resolved

#18461

Check List

  • Functionality includes testing.
  • API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for 164a85e: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for 1825559: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for f16b39f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@github-actions
Copy link
Contributor

❌ Gradle check result for 1741b80: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for 90f73e2: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for ffda262: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for 723ab8f: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

❌ Gradle check result for b54c85e: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

ajleong623 and others added 2 commits July 18, 2025 21:36
Signed-off-by: Anthony Leong <[email protected]>
@ajleong623 ajleong623 marked this pull request as ready for review July 19, 2025 04:41
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

❌ Gradle check result for c1f59b6: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 3, 2025

❌ Gradle check result for 91ca440: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2025

❕ Gradle check result for f033417: UNSTABLE

Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure.

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 5, 2025

❌ Gradle check result for e55207c: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 6, 2025

✅ Gradle check result for 07eeca3: SUCCESS

Signed-off-by: Anthony Leong <[email protected]>
@github-actions
Copy link
Contributor

github-actions bot commented Sep 7, 2025

✅ Gradle check result for c205ff4: SUCCESS

@ajleong623
Copy link
Contributor Author

@msfroh Since I made some changes from the last time you approved, I wanted to confirm if the new changes are still approved. What was happening with serializing the sort field was that the proper source was not being returned. Therefore, I returned the original comparator source and disabled pruning earlier when creating NonPruningSortedSetOrdinalsIndexFieldData. I also added a method to get the underlying delegate SortField for the serialization logic in Lucene

@msfroh msfroh merged commit f0f8078 into opensearch-project:main Sep 10, 2025
31 checks passed
@msfroh msfroh added the v3.3.0 label Sep 10, 2025
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
---------

Signed-off-by: Anthony Leong <[email protected]>
Co-authored-by: HUSTERGS <[email protected]>
Signed-off-by: Ankit Jain <[email protected]>
jainankitk pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 22, 2025
---------

Signed-off-by: Anthony Leong <[email protected]>
Co-authored-by: HUSTERGS <[email protected]>
Signed-off-by: Ankit Jain <[email protected]>
asimmahmood1 pushed a commit to jainankitk/OpenSearch that referenced this pull request Sep 23, 2025
pranikum pushed a commit to pranikum/OpenSearch that referenced this pull request Sep 23, 2025
vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants