-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Add skip_list param for date, scaled float and token count fields #19142
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 skip_list param for date, scaled float and token count fields #19142
Conversation
❌ Gradle check result for 64cae12: 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 beb9dbe: 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 572b134: 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 72c3c39: 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? |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #19142 +/- ##
============================================
- Coverage 72.84% 72.80% -0.04%
+ Complexity 69820 69818 -2
============================================
Files 5674 5674
Lines 320901 320925 +24
Branches 46394 46395 +1
============================================
- Hits 233751 233649 -102
- Misses 68269 68404 +135
+ Partials 18881 18872 -9 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
* follow up to opensearch-project#18889, which added to NumberFieldMapper * remaining are Boolean, which I'm not if its worth adding to * next step is to enable it by default, pending benchmark * update ScaledFloat testRangeQuery to apply same rounding logic to query as indexing to reduce the flakiness Signed-off-by: Asim Mahmood <[email protected]>
183dde7
to
c2ab72a
Compare
❌ Gradle check result for c2ab72a: 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? |
1 similar comment
❌ Gradle check result for c2ab72a: 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? |
❌ Gradle check result for c2ab72a: 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? |
…ensearch-project#19142) Signed-off-by: Asim Mahmood <[email protected]>
Description
Related Issues
Resolves [#17965]
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.