OpenSearch
OpenSearch copied to clipboard
[Backport 2.2] [Bug]: gradle check failing with java heap OutOfMemoryError
Backport 0f2ed704577d6a8a2fa95ee7c110dd7a2b91a141 from #4150
Codecov Report
Merging #4170 (d63df27) into 2.2 (b1017fa) will increase coverage by
0.00%. The diff coverage isn/a.
@@ Coverage Diff @@
## 2.2 #4170 +/- ##
=========================================
Coverage 70.51% 70.52%
- Complexity 56679 56698 +19
=========================================
Files 4557 4557
Lines 272873 272873
Branches 40055 40055
=========================================
+ Hits 192423 192442 +19
+ Misses 64275 64256 -19
Partials 16175 16175
| Impacted Files | Coverage Δ | |
|---|---|---|
| .../java/org/opensearch/node/NodeClosedException.java | 50.00% <0.00%> (-50.00%) |
:arrow_down: |
| ...ava/org/opensearch/action/NoSuchNodeException.java | 0.00% <0.00%> (-50.00%) |
:arrow_down: |
| ...h/action/ingest/SimulateDocumentVerboseResult.java | 60.71% <0.00%> (-39.29%) |
:arrow_down: |
| ...search/aggregations/metrics/ParsedPercentiles.java | 49.47% <0.00%> (-31.58%) |
:arrow_down: |
| ...opensearch/snapshots/SnapshotMissingException.java | 28.57% <0.00%> (-28.58%) |
:arrow_down: |
| ...rc/main/java/org/opensearch/ingest/IngestInfo.java | 51.72% <0.00%> (-27.59%) |
:arrow_down: |
| ...g/opensearch/geometry/utils/StandardValidator.java | 30.30% <0.00%> (-24.25%) |
:arrow_down: |
| ...va/org/opensearch/monitor/process/ProcessInfo.java | 68.00% <0.00%> (-24.00%) |
:arrow_down: |
| .../java/org/opensearch/index/reindex/RemoteInfo.java | 50.45% <0.00%> (-23.86%) |
:arrow_down: |
| ...pensearch/index/shard/SearchOperationListener.java | 72.34% <0.00%> (-23.41%) |
:arrow_down: |
| ... and 478 more |
Help us with your feedback. Take ten seconds to tell us how you rate us.
Gradle Check (Jenkins) Run Completed with:
- RESULT: SUCCESS :white_check_mark:
- URL: https://build.ci.opensearch.org/job/gradle-check/1608/
- CommitID: d63df27e3e83a454fcfb10f7e8ff060275b9dd36
@reta Did you hold off on merging this because of the release?
@reta Did you hold off on merging this because of the release?
thanks @kotwanikunal , not really, was waiting for approval