-
Notifications
You must be signed in to change notification settings - Fork 2.3k
[Pull-based Ingestion] Add ingestion error metrics and make internal queue size configurable #18088
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] Add ingestion error metrics and make internal queue size configurable #18088
Conversation
ab789e4 to
ec1e747
Compare
|
❌ Gradle check result for ec1e747: 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? |
ec1e747 to
43f17d2
Compare
|
❕ Gradle check result for 43f17d2: UNSTABLE Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #18088 +/- ##
============================================
- Coverage 72.57% 72.52% -0.06%
- Complexity 67239 67262 +23
============================================
Files 5476 5476
Lines 310370 310446 +76
Branches 45124 45125 +1
============================================
- Hits 225250 225137 -113
- Misses 66730 66992 +262
+ Partials 18390 18317 -73 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
...stion-kafka/src/internalClusterTest/java/org/opensearch/plugin/kafka/RemoteStoreKafkaIT.java
Show resolved
Hide resolved
server/src/main/java/org/opensearch/indices/pollingingest/DefaultStreamPoller.java
Show resolved
Hide resolved
server/src/main/java/org/opensearch/indices/pollingingest/DefaultStreamPoller.java
Show resolved
Hide resolved
server/src/main/java/org/opensearch/indices/pollingingest/DefaultStreamPoller.java
Outdated
Show resolved
Hide resolved
43f17d2 to
08522b5
Compare
yupeng9
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Signed-off-by: Varun Bharadwaj <[email protected]>
Signed-off-by: Varun Bharadwaj <[email protected]>
08522b5 to
71dd6b9
Compare
|
❌ Gradle check result for 71dd6b9: 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 71dd6b9: 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? |
Description
Sample response:
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.