-
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
[Pull-based Ingestion] Fix snappy related Kafka errors and relax thread leak checks #17240
[Pull-based Ingestion] Fix snappy related Kafka errors and relax thread leak checks #17240
Conversation
❌ Gradle check result for 3d45a2a: 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? |
3d45a2a
to
f988bd8
Compare
❌ Gradle check result for f988bd8: 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? |
f988bd8
to
5b8349a
Compare
❌ Gradle check result for 5b8349a: 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? |
...estion-kafka/src/internalClusterTest/java/org/opensearch/plugin/kafka/IngestFromKafkaIT.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Varun Bharadwaj <[email protected]>
5b8349a
to
fd8dbef
Compare
❌ Gradle check result for fd8dbef: 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? |
Signed-off-by: Varun Bharadwaj <[email protected]>
a8f0a96
to
662fac7
Compare
❌ Gradle check result for 662fac7: 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? |
Signed-off-by: Varun Bharadwaj <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #17240 +/- ##
============================================
+ Coverage 72.24% 72.35% +0.10%
- Complexity 65704 65743 +39
============================================
Files 5318 5318
Lines 305674 305674
Branches 44349 44349
============================================
+ Hits 220834 221161 +327
+ Misses 66769 66408 -361
- Partials 18071 18105 +34 ☔ View full report in Codecov by Sentry. |
...estion-kafka/src/internalClusterTest/java/org/opensearch/plugin/kafka/IngestFromKafkaIT.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Varun Bharadwaj <[email protected]>
Description
This PR is a follow up for pull-based-ingestion to fix the following:
Related Issues
Resolves #17215
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.