Solving wildcard sorting issue for doc_values by ajleong623 · Pull Request #18568 · opensearch-project/OpenSearch

ajleong623 · 2025-06-20T08:37:35Z

Description

Solves the wildcard sorting issue.

Issues Resolved

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 <aj.leong623@gmail.com>

github-actions · 2025-06-20T08:47:51Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-06-26T01:03:09Z

❌ 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?

rest-api-spec/src/main/resources/rest-api-spec/test/search/270_wildcard_fieldtype_queries.yml

github-actions · 2025-07-17T07:46:56Z

❌ 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?

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

github-actions · 2025-07-17T17:32:13Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-07-17T18:54:05Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-07-18T04:49:47Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-07-18T06:24:33Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-07-18T17:48:47Z

❌ 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?

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

Co-authored-by: HUSTERGS <songge.huster@gmail.com> Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

github-actions · 2025-09-05T18:35:28Z

❌ 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 <aj.leong623@gmail.com>

github-actions · 2025-09-06T20:55:35Z

✅ Gradle check result for 07eeca3: SUCCESS

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

github-actions · 2025-09-07T01:52:10Z

✅ Gradle check result for c205ff4: SUCCESS

ajleong623 · 2025-09-09T16:13:02Z

@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

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com>

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com> Signed-off-by: Ankit Jain <jainankitk@apache.org>

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com>

temorary solve

164a85e

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

spotless

1825559

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

opensearch-ci-bot mentioned this pull request Jun 27, 2025

[AUTOCUT] Gradle Check Flaky Test Report for ClientYamlTestSuiteIT #14319

Open

HUSTERGS mentioned this pull request Jul 17, 2025

[BUG] Wildcard sort error when doc_value enabled and size less than total docs #18461

Closed

HUSTERGS reviewed Jul 17, 2025

View reviewed changes

rest-api-spec/src/main/resources/rest-api-spec/test/search/270_wildcard_fieldtype_queries.yml Outdated Show resolved Hide resolved

Merge branch 'opensearch-project:main' into wildcard-bug

f16b39f

still have to change derived field generator

1741b80

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

spotless apply

90f73e2

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

added binary doc values fetcher

ffda262

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

supported field types

723ab8f

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

spotlessapply

b54c85e

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

ajleong623 and others added 2 commits July 18, 2025 21:36

small change

447c45d

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

disable skipping optimization by changing doc values to binary docs

a8a9e84

Co-authored-by: HUSTERGS <songge.huster@gmail.com> Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

ajleong623 marked this pull request as ready for review July 19, 2025 04:41

ajleong623 requested review from Bukhtawar, CEHENKLE, anasalkouz, andrross, ashking94, cwperks, dbwiddis and gbbafna as code owners July 19, 2025 04:41

retry for flaky

e55207c

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

retry for flaky

07eeca3

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

retry for flaky

c205ff4

Signed-off-by: Anthony Leong <aj.leong623@gmail.com>

msfroh approved these changes Sep 10, 2025

View reviewed changes

msfroh merged commit f0f8078 into opensearch-project:main Sep 10, 2025
31 checks passed

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

Solving wildcard sorting issue for doc_values (opensearch-project#18568)

5c78e74

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com>

pranikum pushed a commit to pranikum/OpenSearch that referenced this pull request Sep 23, 2025

Solving wildcard sorting issue for doc_values (opensearch-project#18568)

9dabdd5

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com>

opensearch-ci-bot mentioned this pull request Sep 25, 2025

[AUTOCUT] Gradle Check Flaky Test Report for TelemetryMetricsEnabledSanityIT #19422

Open

vinaykpud pushed a commit to vinaykpud/OpenSearch that referenced this pull request Sep 26, 2025

Solving wildcard sorting issue for doc_values (opensearch-project#18568)

3334bc8

--------- Signed-off-by: Anthony Leong <aj.leong623@gmail.com> Co-authored-by: HUSTERGS <songge.huster@gmail.com>

BrewTestBot mentioned this pull request Oct 15, 2025

opensearch 3.3.0 Homebrew/homebrew-core#249627

Merged

opensearch-ci-bot mentioned this pull request Nov 9, 2025

[AUTOCUT] Gradle Check Flaky Test Report for MultiTermsAggregatorTests #19845

Open

This was referenced Jan 10, 2026

docs: add wildcard-field-mapper report for v3.3.0 tkykenmt/opensearch-feature-explorer#1494

Merged

[feature] Wildcard Field Mapper tkykenmt/opensearch-feature-explorer#1415

Closed

opensearch-ci-bot mentioned this pull request Feb 18, 2026

[AUTOCUT] Gradle Check Flaky Test Report for ExampleFixtureIT #20657

Open

opensearch-ci-bot mentioned this pull request Mar 9, 2026

[AUTOCUT] Gradle Check Flaky Test Report for AzureStorageCleanupThirdPartyTests #19037

Open

Conversation

ajleong623 commented Jun 20, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Issues Resolved

Check List

Uh oh!

github-actions bot commented Jun 20, 2025

Uh oh!

github-actions bot commented Jun 26, 2025

Uh oh!

Uh oh!

github-actions bot commented Jul 17, 2025

Uh oh!

github-actions bot commented Jul 17, 2025

Uh oh!

github-actions bot commented Jul 17, 2025

Uh oh!

github-actions bot commented Jul 18, 2025

Uh oh!

github-actions bot commented Jul 18, 2025

Uh oh!

github-actions bot commented Jul 18, 2025

Uh oh!

github-actions bot commented Sep 5, 2025

Uh oh!

github-actions bot commented Sep 6, 2025

Uh oh!

github-actions bot commented Sep 7, 2025

Uh oh!

ajleong623 commented Sep 9, 2025

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

ajleong623 commented Jun 20, 2025 •

edited

Loading