-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
[Backport 2.x] [Feature] Visitor design pattern in QueryBuilder #10167
[Backport 2.x] [Feature] Visitor design pattern in QueryBuilder #10167
Conversation
Signed-off-by: Varun Jain <varunudr@amazon.com>
Hey @vibrantvarun, rebase for conflicts? |
Signed-off-by: Varun Jain <varunudr@amazon.com>
Compatibility status:Checks if related components are compatible with change cf10d30 Incompatible componentsSkipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/performance-analyzer-rca.git] |
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## 2.x #10167 +/- ##
============================================
+ Coverage 70.83% 70.93% +0.09%
- Complexity 58250 58353 +103
============================================
Files 4811 4812 +1
Lines 275316 275404 +88
Branches 40464 40485 +21
============================================
+ Hits 195033 195360 +327
+ Misses 63576 63349 -227
+ Partials 16707 16695 -12
|
Gradle Check (Jenkins) Run Completed with:
|
Description
Backport 10110 to 2.x
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
9986
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.