-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Fix regex query to work with field alias #18215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix regex query to work with field alias #18215
Conversation
❌ Gradle check result for d4de07c: 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: bowenlan-amzn <[email protected]>
❌ Gradle check result for 8ce9cc1: 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? |
❌ Gradle check result for 6063867: 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? |
❌ Gradle check result for d9afc10: 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? |
❌ Gradle check result for eb6c89c: 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: bowenlan-amzn <[email protected]>
❌ Gradle check result for 5ee9aa4: 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: bowenlan-amzn <[email protected]>
❌ Gradle check result for 9dec052: 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: bowenlan-amzn <[email protected]>
❕ Gradle check result for 9e680a1: 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. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #18215 +/- ##
============================================
+ Coverage 72.80% 72.84% +0.04%
- Complexity 68437 68479 +42
============================================
Files 5563 5563
Lines 314174 314176 +2
Branches 45554 45555 +1
============================================
+ Hits 228726 228854 +128
+ Misses 66871 66782 -89
+ Partials 18577 18540 -37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
❌ Gradle check result for 1a2e35f: 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? |
❌ Gradle check result for d6a0948: 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? |
❌ Gradle check result for d6a0948: 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? |
❌ Gradle check result for d6a0948: 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? |
❌ Gradle check result for 6c5d95d: null 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: Ankit Jain <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
❌ Gradle check result for 608a1c2: 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? |
❌ Gradle check result for 608a1c2: 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: bowenlan-amzn <[email protected]> Signed-off-by: Ankit Jain <[email protected]> Co-authored-by: Ankit Jain <[email protected]>
* Fix regex query to work with field alias (opensearch-project#18215) --------- Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: Ankit Jain <[email protected]> Co-authored-by: Ankit Jain <[email protected]> * Add workflow to automatically trigger manifest generation process (opensearch-project#18629) Signed-off-by: Sayali Gaikawad <[email protected]> * Bump com.squareup.okio:okio from 3.13.0 to 3.14.0 in /test/fixtures/hdfs-fixture (opensearch-project#18645) * Bump com.squareup.okio:okio in /test/fixtures/hdfs-fixture Bumps [com.squareup.okio:okio](https://github.com/square/okio) from 3.13.0 to 3.14.0. - [Release notes](https://github.com/square/okio/releases) - [Changelog](https://github.com/square/okio/blob/master/CHANGELOG.md) - [Commits](square/okio@parent-3.13.0...parent-3.14.0) --- updated-dependencies: - dependency-name: com.squareup.okio:okio dependency-version: 3.14.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix credentials format (opensearch-project#18651) Signed-off-by: Sayali Gaikawad <[email protected]> * Provide aux transport to SecureAuxTransportSettingsProvider (opensearch-project#18616) * Pass aux transport to secure settings parameters(). Signed-off-by: Finn Carroll <[email protected]> * Update javadocs, fix UTs. Signed-off-by: Finn Carroll <[email protected]> * Spotless apply Signed-off-by: Finn Carroll <[email protected]> * Pass aux transport key as unique identifier instead of transport ref. Signed-off-by: Finn Carroll <[email protected]> * Consistent naming. auxTransportSettingKey -> auxTransportType. Signed-off-by: Finn Carroll <[email protected]> * Changelog. Signed-off-by: Finn Carroll <[email protected]> * Spotless apply Signed-off-by: Finn Carroll <[email protected]> --------- Signed-off-by: Finn Carroll <[email protected]> * Making snapshot store/restore rate dynamic (opensearch-project#18069) * Making snapshot store/restore rate dynamic Signed-off-by: kkewwei <[email protected]> Signed-off-by: kkewwei <[email protected]> * simple the code Signed-off-by: kkewwei <[email protected]> Signed-off-by: kkewwei <[email protected]> --------- Signed-off-by: kkewwei <[email protected]> Signed-off-by: kkewwei <[email protected]> * Bump com.netflix.nebula.ospackage-base from 11.11.2 to 12.0.0 in /distribution/packages (opensearch-project#18646) * Bump com.netflix.nebula.ospackage-base in /distribution/packages Bumps com.netflix.nebula.ospackage-base from 11.11.2 to 12.0.0. --- updated-dependencies: - dependency-name: com.netflix.nebula.ospackage-base dependency-version: 12.0.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Craig Perkins <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Craig Perkins <[email protected]> * Extend Profiler to allow for non-timing info (opensearch-project#18631) * copied from other branch, ready for pr Signed-off-by: TJ Neuenfeldt <[email protected]> * cleanup from merging Signed-off-by: TJ Neuenfeldt <[email protected]> * another spotless apply and fixed breaking change Signed-off-by: TJ Neuenfeldt <[email protected]> * another spotlessApply Signed-off-by: TJ Neuenfeldt <[email protected]> * forgot to change back concurrent agg stuff Signed-off-by: TJ Neuenfeldt <[email protected]> * forgot to spotless after prev fix Signed-off-by: TJ Neuenfeldt <[email protected]> * added more unit tests for coverage Signed-off-by: TJ Neuenfeldt <[email protected]> * changed to supplier, removed timer ctors Signed-off-by: TJ Neuenfeldt <[email protected]> * made feedback changes Signed-off-by: TJ Neuenfeldt <[email protected]> * forgot javadoc for new class Signed-off-by: TJ Neuenfeldt <[email protected]> * forgot to revert name to toString Signed-off-by: TJ Neuenfeldt <[email protected]> * fix count check in concurrentquerypb Signed-off-by: TJ Neuenfeldt <[email protected]> * unmodifiable map add Signed-off-by: TJ Neuenfeldt <[email protected]> --------- Signed-off-by: TJ Neuenfeldt <[email protected]> Signed-off-by: TJ Neuenfeldt <[email protected]> * [Bug Fix] Fix the backward compatibility regression with `COMPLEMENT` for Regexp queries introduced in OpenSearch 3.0 (opensearch-project#18640) * FIx ~ bug Signed-off-by: Prudhvi Godithi <[email protected]> * Update changelog Signed-off-by: Prudhvi Godithi <[email protected]> --------- Signed-off-by: Prudhvi Godithi <[email protected]> * [WLM] Make resource usage value cacheable for NodeDuressTrackers (opensearch-project#18649) --------- Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Ankit Jain <[email protected]> Co-authored-by: Ankit Jain <[email protected]> * Fix bugs in replication lag computation (opensearch-project#18602) * Fix bug in replication lag computation. This change fixes a bug with replication lag computation to correctly use epoch reference point with Instant.now() and DateUtils. This change also fixes pruning logic to correctly remove the latest synced to checkpoint from tracking. Previously we would only prune up to the latest. This ensures that when a new checkpoint is eventually received we aren't incorrectly computing lag from the synced-to checkpoint. Signed-off-by: Marc Handalian <[email protected]> * add changelog entry Signed-off-by: Marc Handalian <[email protected]> * Fix flaky test to hardcode time lag between checkpoint arrival. Signed-off-by: Marc Handalian <[email protected]> --------- Signed-off-by: Marc Handalian <[email protected]> * Replace LegacyESVersion.fromString with Version.fromString (opensearch-project#18567) --------- Signed-off-by: Du Tran <[email protected]> * Inject QueryCollectorSpecFactory from Plugins to create QueryCollectorContext in QueryPhase (opensearch-project#18637) * Initial Commit Signed-off-by: Varun Jain <[email protected]> * Javadocs Signed-off-by: Varun Jain <[email protected]> * Factory design pattern for QueryCollectorContextSpec Signed-off-by: Varun Jain <[email protected]> * Fixing Concurrent Segment Search Signed-off-by: Varun Jain <[email protected]> * Remove spec Signed-off-by: Varun Jain <[email protected]> * Renaming method Signed-off-by: Varun Jain <[email protected]> * Javadoc Signed-off-by: Varun Jain <[email protected]> * Removing support method from QueryCollectorContextSpecFactory Signed-off-by: Varun Jain <[email protected]> * Adding javadoc Signed-off-by: Varun Jain <[email protected]> * Add changelog Signed-off-by: Varun Jain <[email protected]> * Add experimentalAPI annotation Signed-off-by: Varun Jain <[email protected]> * remove redundant check from getQueryCollectorContextSpec method Signed-off-by: Varun Jain <[email protected]> * Add unit tests Signed-off-by: Varun Jain <[email protected]> * Remove Test annotation Signed-off-by: Varun Jain <[email protected]> * Reformatting Signed-off-by: Varun Jain <[email protected]> --------- Signed-off-by: Varun Jain <[email protected]> * Bump com.azure:azure-storage-blob from 12.30.0 to 12.30.1 in /plugins/repository-azure (opensearch-project#18644) * Bump com.azure:azure-storage-blob in /plugins/repository-azure Bumps [com.azure:azure-storage-blob](https://github.com/Azure/azure-sdk-for-java) from 12.30.0 to 12.30.1. - [Release notes](https://github.com/Azure/azure-sdk-for-java/releases) - [Commits](Azure/azure-sdk-for-java@azure-storage-blob_12.30.0...azure-storage-blob_12.30.1) --- updated-dependencies: - dependency-name: com.azure:azure-storage-blob dependency-version: 12.30.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Updating SHAs Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Fix checkpoint handling to prevent segment replication infinite loop (opensearch-project#18636) * Fix checkpoint handling to prevent segment replication infinite loop Signed-off-by: Ashish Singh <[email protected]> * Modify existing test to verify code change behaviour Signed-off-by: Ashish Singh <[email protected]> --------- Signed-off-by: Ashish Singh <[email protected]> * Extend Approximation Framework to other numeric types (opensearch-project#18530) * Extend approximation to other numeric types Signed-off-by: Prudhvi Godithi <[email protected]> * Fix basic tests Signed-off-by: Prudhvi Godithi <[email protected]> * Fix basic tests Signed-off-by: Prudhvi Godithi <[email protected]> * Add Parameterized tests for all numeric types Signed-off-by: Prudhvi Godithi <[email protected]> * Fix FieldSortIT Signed-off-by: Prudhvi Godithi <[email protected]> * Fix FieldSortIT Signed-off-by: Prudhvi Godithi <[email protected]> * Address comments Signed-off-by: Prudhvi Godithi <[email protected]> * Update tests and changelog Signed-off-by: Prudhvi Godithi <[email protected]> * Update tests and changelog Signed-off-by: Prudhvi Godithi <[email protected]> --------- Signed-off-by: Prudhvi Godithi <[email protected]> * Bump BouncyCastle and password4j (opensearch-project#18668) Changes: - Bump org.bouncycastle:bctls-fips from 2.0.19 to 2.0.20 - Bump org.bouncycastle:bcpkix-fips from 2.0.7 to 2.0.8 - Bump org.bouncycastle:bcpg-fips from 2.0.10 to 2.0.11 - Bump com.password4j:password4j from 1.8.2 to 1.8.3 Signed-off-by: Andrey Pleskach <[email protected]> * Add support for custom index name resolver from cluster plugin (opensearch-project#18593) Signed-off-by: Gagan Singh Saini <[email protected]> * Remove clone holder and add Cleaner logic to clean up clones in FullFileCachedIndexInput (opensearch-project#18662) Signed-off-by: Shreyansh Ray <[email protected]> * Disallow resize for Warm Index, add Parameterized ITs for close in remote store (opensearch-project#18686) Signed-off-by: Gagan Singh Saini <[email protected]> Co-authored-by: Gaurav Bafna <[email protected]> * Added support for range version support in semver (opensearch-project#18557) * Added support for range version support in semver Signed-off-by: Shruti Garg <[email protected]> * Added test coverage for missing lines Signed-off-by: Shruti Garg <[email protected]> * Fixed for non reachable code line and tests Signed-off-by: Shruti Garg <[email protected]> --------- Signed-off-by: Shruti Garg <[email protected]> * An attempt to fix flaky SecureReactorNetty4HttpServerTransportTests (opensearch-project#18680) Signed-off-by: Andriy Redko <[email protected]> * Bump com.squareup.okio:okio from 3.14.0 to 3.15.0 in /test/fixtures/hdfs-fixture (opensearch-project#18689) * Bump com.squareup.okio:okio in /test/fixtures/hdfs-fixture Bumps [com.squareup.okio:okio](https://github.com/square/okio) from 3.14.0 to 3.15.0. - [Release notes](https://github.com/square/okio/releases) - [Changelog](https://github.com/square/okio/blob/master/CHANGELOG.md) - [Commits](square/okio@parent-3.14.0...parent-3.15.0) --- updated-dependencies: - dependency-name: com.squareup.okio:okio dependency-version: 3.15.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Add Semantic Version field type mapper (opensearch-project#18454) * Add Semantic Version field type mapper and extensive unit tests Signed-off-by: Siddhant Deshmukh <[email protected]> * Refactor SemanticVersionFieldMapper for flexible index, store, and doc_values support Signed-off-by: Siddhant Deshmukh <[email protected]> --------- Signed-off-by: Siddhant Deshmukh <[email protected]> * Bump com.azure:azure-core from 1.55.3 to 1.55.5 in /plugins/repository-azure (opensearch-project#18691) * Bump com.azure:azure-core in /plugins/repository-azure Bumps [com.azure:azure-core](https://github.com/Azure/azure-sdk-for-java) from 1.55.3 to 1.55.5. - [Release notes](https://github.com/Azure/azure-sdk-for-java/releases) - [Commits](Azure/azure-sdk-for-java@azure-core_1.55.3...azure-core_1.55.5) --- updated-dependencies: - dependency-name: com.azure:azure-core dependency-version: 1.55.5 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Updating SHAs Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Exposing default codec from codec service (opensearch-project#18698) * Exposing default codec from codec service Signed-off-by: bharath-techie <[email protected]> * addressing comments Signed-off-by: bharath-techie <[email protected]> --------- Signed-off-by: bharath-techie <[email protected]> * Replace CPU load average logic with AverageTracker class and modify default thresholds (opensearch-project#18666) Signed-off-by: Harsh Kothari <[email protected]> * Rename all query group references to workload group (opensearch-project#18711) Signed-off-by: Ruirui Zhang <[email protected]> * Rename WorkloadGroupTestUtil to WorkloadManagementTestUtil (opensearch-project#18709) Signed-off-by: Ruirui Zhang <[email protected]> * Add Prudhvi Godithi (GH: prudhvigodithi) as a maintainer (opensearch-project#18690) * Add Prudhvi Godithi (GH: prudhvigodithi) as a maintainer Signed-off-by: Craig Perkins <[email protected]> * Keep spaces Signed-off-by: Craig Perkins <[email protected]> * Fix spacing Signed-off-by: Craig Perkins <[email protected]> --------- Signed-off-by: Craig Perkins <[email protected]> * Pass index settings to system ingest processor factories. (opensearch-project#18708) Signed-off-by: Bo Zhang <[email protected]> * Include named queries from rescore contexts in matched_queries array (opensearch-project#18697) * Fix: Include named queries from rescore contexts in matched_queries array - Modified QueryRescoreContext to store ParsedQuery instead of just Query - Updated QueryRescorerBuilder to use context.toQuery() for capturing named queries - Enhanced MatchedQueriesPhase to collect named queries from all rescore contexts - Added comprehensive tests for both unit and REST API scenarios - Resolves inconsistency with Elasticsearch behavior where rescore named queries were not surfaced - Cleaned up unnecessary comments for better code readability Signed-off-by: Girish Jeyakumar <[email protected]> * Address maintainer feedback: update RescoreContext interface design - Added parsedQuery() method to RescoreContext interface - Added getParsedQueries() method to RescoreContext interface - Updated QueryRescoreContext to override interface methods - Removed instanceof check in MatchedQueriesPhase by using interface method - Simplified assignment in QueryRescorerBuilder to one line Signed-off-by: Girish Jeyakumar <[email protected]> * Complete interface replacement: replace query() with parsedQuery() and getQueries() with getParsedQueries() - Removed query() method from QueryRescoreContext - Removed getQueries() method from RescoreContext interface - Updated all callers to use parsedQuery().query() instead of query() - Updated DfsPhase to use getParsedQueries() instead of getQueries() - Updated QueryRescorer to use parsedQuery().query() for rescoring and explanations - Updated QueryRescorerBuilderTests to use parsedQuery().query() - This addresses the maintainer's feedback for a cleaner interface design Signed-off-by: Girish Jeyakumar <[email protected]> * Finalize interface design and move changelog entry to Added section - Removed parsedQuery() method from RescoreContext base interface - Renamed getParsedQuery() to parsedQuery() in QueryRescoreContext for consistency - Updated all callers to use parsedQuery() method - Moved changelog entry to 'Added' section (more appropriate than 'Fixed') - Added proper imports for ParsedQuery to improve code readability - All tests pass and code formatting is correct Final interface design: - RescoreContext: only getParsedQueries() (returns List<ParsedQuery>) - QueryRescoreContext: parsedQuery() + getParsedQueries() override - MatchedQueriesPhase: uses getParsedQueries().forEach() for clean iteration This addresses all maintainer feedback from @mch2 and provides a clean, well-tested implementation that surfaces named queries from rescore contexts in the matched_queries array. Signed-off-by: Girish Jeyakumar <[email protected]> * Maintain backward compatibility for RescoreContext public API - Restore getQueries() method for backward compatibility - Add getParsedQueries() method for new functionality Signed-off-by: Girish Jeyakumar <[email protected]> * Fix mixed cluster test failure by updating skip version to 3.2.0 Signed-off-by: girish jeyakumar <[email protected]> --------- Signed-off-by: Girish Jeyakumar <[email protected]> Signed-off-by: girish jeyakumar <[email protected]> Signed-off-by: Marc Handalian <[email protected]> Co-authored-by: Girish Jeyakumar <[email protected]> Co-authored-by: Marc Handalian <[email protected]> * Move gRPC transitive dependencies to expected version (opensearch-project#18672) * Some explicit transient gRPC deps are not the expected version. grpc-netty-shaded:1.68.2 expects perfmark-api:0.27.0 guava:33.2.1-jre expects failureaccess:1.0.2 Signed-off-by: Finn Carroll <[email protected]> * Update license shas. Signed-off-by: Finn Carroll <[email protected]> * Changelog. Signed-off-by: Finn Carroll <[email protected]> * Rebase changelog. Signed-off-by: Finn Carroll <[email protected]> --------- Signed-off-by: Finn Carroll <[email protected]> * Modify wlm logging message (opensearch-project#18712) Signed-off-by: Ruirui Zhang <[email protected]> * [WLM] Introduce rule cardinality check (opensearch-project#18663) Signed-off-by: Kaushal Kumar <[email protected]> * [WLM] Add WLM mode validation for workload group CRUD requests (opensearch-project#18652) Signed-off-by: Ruirui Zhang <[email protected]> * Allow upload of translog-1 generation while migrating to remote (opensearch-project#18713) Signed-off-by: Gaurav Bafna <[email protected]> * Fix max_score is null when using _score as a primary sort (opensearch-project#18715) * Fix max_score is null when sorting on score firstly Signed-off-by: Binlong Gao <[email protected]> * modify changelog Signed-off-by: Binlong Gao <[email protected]> * Fix test failure Signed-off-by: Binlong Gao <[email protected]> --------- Signed-off-by: Binlong Gao <[email protected]> * Changes for the Upload Workflow for merged segments (opensearch-project#18610) * Changes for the Upload Workflow for merged segments Signed-off-by: Aditi Goyal <[email protected]> * Addressed Todos Signed-off-by: Aditi Goyal <[email protected]> * Added Test Cases Signed-off-by: Aditi Goyal <[email protected]> * Removed the usage of Forbidden APIs Signed-off-by: Aditi Goyal <[email protected]> * Updated the code to include priority upload as paramter Signed-off-by: Aditi Goyal <[email protected]> * Updated the remote listener to use remote upploader interface Signed-off-by: Aditi Goyal <[email protected]> * Added JavaDocs across the changes Signed-off-by: Aditi Goyal <[email protected]> * Addressed spotless check Signed-off-by: Aditi Goyal <[email protected]> * Dummy commit(can be reverted) Signed-off-by: Aditi Goyal <[email protected]> * Revert "Dummy commit(can be reverted)" This reverts commit e542b99. Signed-off-by: Aditi Goyal <[email protected]> --------- Signed-off-by: Aditi Goyal <[email protected]> Co-authored-by: Aditi Goyal <[email protected]> * Modify merged segment pre-copy classes licenses (opensearch-project#18737) Signed-off-by: guojialiang <[email protected]> * `Lucene.java` Code Improvement (opensearch-project#18710) * Code Improvement Signed-off-by: Prudhvi Godithi <[email protected]> * Code Improvement with a new method Signed-off-by: Prudhvi Godithi <[email protected]> --------- Signed-off-by: Prudhvi Godithi <[email protected]> * Bump org.apache.commons:commons-lang3 from 3.17.0 to 3.18.0 in /plugins/repository-hdfs (opensearch-project#18745) * Bump org.apache.commons:commons-lang3 in /plugins/repository-hdfs Bumps org.apache.commons:commons-lang3 from 3.17.0 to 3.18.0. --- updated-dependencies: - dependency-name: org.apache.commons:commons-lang3 dependency-version: 3.18.0 dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * Updating SHAs Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump org.apache.logging.log4j:log4j-core from 2.25.0 to 2.25.1 in /buildSrc/src/testKit/thirdPartyAudit/sample_jars (opensearch-project#18744) * Bump org.apache.logging.log4j:log4j-core Bumps org.apache.logging.log4j:log4j-core from 2.25.0 to 2.25.1. --- updated-dependencies: - dependency-name: org.apache.logging.log4j:log4j-core dependency-version: 2.25.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * Bump com.azure:azure-storage-common from 12.29.0 to 12.29.1 in /plugins/repository-azure (opensearch-project#18742) * Bump com.azure:azure-storage-common in /plugins/repository-azure Bumps [com.azure:azure-storage-common](https://github.com/Azure/azure-sdk-for-java) from 12.29.0 to 12.29.1. - [Release notes](https://github.com/Azure/azure-sdk-for-java/releases) - [Commits](Azure/azure-sdk-for-java@azure-storage-blob_12.29.0...azure-storage-blob_12.29.1) --- updated-dependencies: - dependency-name: com.azure:azure-storage-common dependency-version: 12.29.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Updating SHAs Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: gaobinlong <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: gaobinlong <[email protected]> * Bump com.google.jimfs:jimfs from 1.3.0 to 1.3.1 in /distribution/tools/plugin-cli (opensearch-project#18743) * Bump com.google.jimfs:jimfs in /distribution/tools/plugin-cli Bumps [com.google.jimfs:jimfs](https://github.com/google/jimfs) from 1.3.0 to 1.3.1. - [Release notes](https://github.com/google/jimfs/releases) - [Commits](google/jimfs@v1.3.0...v1.3.1) --- updated-dependencies: - dependency-name: com.google.jimfs:jimfs dependency-version: 1.3.1 dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * Update changelog Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: gaobinlong <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: gaobinlong <[email protected]> * Fix sync method to account for block files in Composite Directory (opensearch-project#18660) Signed-off-by: Shreyansh Ray <[email protected]> * [Cleanup] Remove upgrade-cli tool and related build references (opensearch-project#18494) The `upgrade-cli` tool is deprecated and no longer needed. - Removed: distribution/tools/upgrade-cli - Cleaned up: settings.gradle, distribution/build.gradle - Deleted: distribution/src/bin/opensearch-upgrade(.bat) Signed-off-by: BeomSeogKim <[email protected]> * Improve test instructions in developer guide (opensearch-project#18736) Signed-off-by: Shahbaz Aamir <[email protected]> * [Experimental] Start without joining a cluster if a "clusterless" ClusterPlugin is loaded (opensearch-project#18479) These are the core changes to allow ClusterPlugins to manage cluster state on behalf of an OpenSearch node. If a "clusterless" ClusterPlugin is loaded, then the node will start up with dummy versions of Discovery and ClusterManagerService, and will load the minimum cluster state required for startup (essentially, the node needs to see itself). From there, the ClusterPlugin can interact with ClusterApplierService to apply some cluster state to the node. (The details of constructing/receiving that cluster state is up to the plugin.) --------- Signed-off-by: Michael Froh <[email protected]> * Check CircuitBreaker before put/compute in FileCache to avoid entry removal (opensearch-project#18661) Signed-off-by: Shreyansh Ray <[email protected]> * Fix UNIX domain socket permission checks on Windows (opensearch-project#18764) Signed-off-by: Andriy Redko <[email protected]> --------- Signed-off-by: bowenlan-amzn <[email protected]> Signed-off-by: Ankit Jain <[email protected]> Signed-off-by: Sayali Gaikawad <[email protected]> Signed-off-by: dependabot[bot] <[email protected]> Signed-off-by: Finn Carroll <[email protected]> Signed-off-by: kkewwei <[email protected]> Signed-off-by: kkewwei <[email protected]> Signed-off-by: Craig Perkins <[email protected]> Signed-off-by: TJ Neuenfeldt <[email protected]> Signed-off-by: TJ Neuenfeldt <[email protected]> Signed-off-by: Prudhvi Godithi <[email protected]> Signed-off-by: Kaushal Kumar <[email protected]> Signed-off-by: Marc Handalian <[email protected]> Signed-off-by: Du Tran <[email protected]> Signed-off-by: Varun Jain <[email protected]> Signed-off-by: Ashish Singh <[email protected]> Signed-off-by: Andrey Pleskach <[email protected]> Signed-off-by: Gagan Singh Saini <[email protected]> Signed-off-by: Shreyansh Ray <[email protected]> Signed-off-by: Shruti Garg <[email protected]> Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: Siddhant Deshmukh <[email protected]> Signed-off-by: bharath-techie <[email protected]> Signed-off-by: Harsh Kothari <[email protected]> Signed-off-by: Ruirui Zhang <[email protected]> Signed-off-by: Bo Zhang <[email protected]> Signed-off-by: Girish Jeyakumar <[email protected]> Signed-off-by: girish jeyakumar <[email protected]> Signed-off-by: Gaurav Bafna <[email protected]> Signed-off-by: Binlong Gao <[email protected]> Signed-off-by: Aditi Goyal <[email protected]> Signed-off-by: guojialiang <[email protected]> Signed-off-by: gaobinlong <[email protected]> Signed-off-by: BeomSeogKim <[email protected]> Signed-off-by: Shahbaz Aamir <[email protected]> Signed-off-by: Michael Froh <[email protected]> Co-authored-by: bowenlan-amzn <[email protected]> Co-authored-by: Ankit Jain <[email protected]> Co-authored-by: Sayali Gaikawad <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Finn <[email protected]> Co-authored-by: kkewwei <[email protected]> Co-authored-by: Craig Perkins <[email protected]> Co-authored-by: TJ Neuenfeldt <[email protected]> Co-authored-by: Prudhvi Godithi <[email protected]> Co-authored-by: Kaushal Kumar <[email protected]> Co-authored-by: Marc Handalian <[email protected]> Co-authored-by: Trần Quang Dự <[email protected]> Co-authored-by: Varun Jain <[email protected]> Co-authored-by: Ashish Singh <[email protected]> Co-authored-by: Andrey Pleskach <[email protected]> Co-authored-by: Gagan6164 <[email protected]> Co-authored-by: rayshrey <[email protected]> Co-authored-by: Gaurav Bafna <[email protected]> Co-authored-by: shrugarg-amzn <[email protected]> Co-authored-by: Andriy Redko <[email protected]> Co-authored-by: Siddhant Deshmukh <[email protected]> Co-authored-by: Bharathwaj G <[email protected]> Co-authored-by: Harsh Kothari <[email protected]> Co-authored-by: Ruirui Zhang <[email protected]> Co-authored-by: Bo Zhang <[email protected]> Co-authored-by: Girish Jeyakumar <[email protected]> Co-authored-by: Girish Jeyakumar <[email protected]> Co-authored-by: Gaurav Bafna <[email protected]> Co-authored-by: gaobinlong <[email protected]> Co-authored-by: Aditi Goyal <[email protected]> Co-authored-by: Aditi Goyal <[email protected]> Co-authored-by: guojialiang <[email protected]> Co-authored-by: Adrian-Kim <[email protected]> Co-authored-by: Shahbaz Aamir <[email protected]> Co-authored-by: Michael Froh <[email protected]>
Description
I spot a history change that seems change the logic a bit when building regex query
https://github.com/opensearch-project/OpenSearch/pull/3967/files
Revert that logic change essentially fix the problem.
The reason why alias field map is not effective before is because
super.getRegexpQuery
which goes to lucene that have no idea of alias field.We have to use
currentFieldType
resolved fromqueryBuilder.context.fieldMapper(field)
that knows about field alias.Related Issues
Resolves #18214
Check List
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.