Update to account for XContent refactor in 2.x #1485
Conversation
Xcontent classes were moved in this PR opensearch-project/OpenSearch#5902 causing builds to fail. --------- Signed-off-by: MaxKsyunz <maxk@bitquilltech.com> (cherry picked from commit cd95bc5)
Signed-off-by: MaxKsyunz <maxk@bitquilltech.com>
MaxKsyunz
changed the title
Codecov Report
📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more @@ Coverage Diff @@
## 2.x #1485 +/- ##
============================================
- Coverage 99.96% 98.46% -1.50%
- Complexity 2593 3867 +1274
============================================
Files 220 345 +125
Lines 6205 9600 +3395
Branches 378 616 +238
============================================
+ Hits 6203 9453 +3250
- Misses 2 142 +140
- Partials 0 5 +5
Flags with carried forward coverage won't be shown. Click here to find out more. see 125 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
integ-test/src/test/java/org/opensearch/sql/legacy/RestIntegTestCase.java
Show resolved
Hide resolved
build.gradle
Outdated
| @@ -98,6 +98,8 @@ subprojects { | |||
| mavenLocal() | |||
| maven { url "https://aws.oss.sonatype.org/content/repositories/snapshots" } | |||
| mavenCentral() | |||
| // todo. remove this when lucene 9.4.0 is released | |||
| maven { url "https://d1nvenhzbhpy0q.cloudfront.net/snapshots/lucene/" } | |||
There was a problem hiding this comment.
Good catch! I'll remove it -- it does not apply to 2.x branch.
It was added when main was bumped to 3.0.
The PR to main moved that line because the build was picking up dependencies from that random repository instead of sonatype and failing.
Signed-off-by: MaxKsyunz <maxk@bitquilltech.com>
MaxKsyunz
dismissed stale reviews from derek-ho, rupal-bq, and Yury-Fridlyand
via
7d3793b
Description
Cherry pick of cd95bc5 plus fixes for 2.x-only changes.
Issues Resolved
#1475
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.