-
Notifications
You must be signed in to change notification settings - Fork 0
Add Support for Alternate Legacy MULTIMATCH syntax #168
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
Add Support for Alternate Legacy MULTIMATCH syntax #168
Conversation
2f8e6ed to
ea0f277
Compare
sql/src/main/java/org/opensearch/sql/sql/parser/AstExpressionBuilder.java
Outdated
Show resolved
Hide resolved
sql/src/main/java/org/opensearch/sql/sql/parser/AstExpressionBuilder.java
Outdated
Show resolved
Hide resolved
core/src/main/java/org/opensearch/sql/expression/function/OpenSearchFunctions.java
Outdated
Show resolved
Hide resolved
be6214c to
6e0b326
Compare
Codecov Report
@@ Coverage Diff @@
## integ-add-alternate-multi-match-syntax #168 +/- ##
=========================================================================
Coverage 98.29% 98.29%
Complexity 3465 3465
=========================================================================
Files 347 347
Lines 8645 8645
Branches 550 550
=========================================================================
Hits 8498 8498
Misses 142 142
Partials 5 5
Flags with carried forward coverage won't be shown. Click here to find out more. 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
6e0b326 to
42ba4fd
Compare
d4336d2 to
8f80e99
Compare
integ-test/src/test/java/org/opensearch/sql/sql/MultiMatchIT.java
Outdated
Show resolved
Hide resolved
8f80e99 to
a722d51
Compare
...src/test/java/org/opensearch/sql/opensearch/storage/script/filter/lucene/MultiMatchTest.java
Outdated
Show resolved
Hide resolved
a722d51 to
f1a51e8
Compare
Signed-off-by: forestmvey <forestv@bitquilltech.com>
f1a51e8 to
b56025c
Compare
Signed-off-by: Heemin Kim <heemin@amazon.com>
* Import h3 library (#154) Made following changes to make it compatible: 1. Rename package from elasticsearch to opensearch.geospatial 2. Update License headers 3. Update build file 4. Update settings to include sub projects * Use Transport Request (#164) Remove usage of deprecated BaseNodeRequest * Update http client package to resolve build failure (#168) (#171) * Introduce H3 min resolution constant (#165) H3 version 1 has 16 resolutions, numbered 0 through 15. Introduced a constant to represent min value, similar to max value. * Add geohex aggregation (#160) This aggregation will use uber's h3 to group coordinates into H3 cell. Created new aggregation type geohex_grid. The precision will be between 0 and 15. This aggreation has default precision as 5, similar to geohash and geotile. Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com> * Add integration test (#176) Included integration test for geohex_grid. Signed-off-by: Vijayan Balasubramanian <balasvij@amazon.com>
Signed-off-by: forestmvey forestv@bitquilltech.com
Description
Add support for alternative syntax of
MULTI_MATCHrelevance search functions. Legacy engine supports parameter keys as quoted, or unquoted values for both required and optional arguments. This syntax has been ported over to all relevance functions in the V2 engine.Example Queries
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.