-
Notifications
You must be signed in to change notification settings - Fork 109
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
Consumer tracer to replace the use of Vert.x Kafka lib in base dispatcher module. #3269
Consumer tracer to replace the use of Vert.x Kafka lib in base dispatcher module. #3269
Conversation
Codecov Report
@@ Coverage Diff @@
## main #3269 +/- ##
============================================
- Coverage 61.89% 61.67% -0.22%
- Complexity 765 767 +2
============================================
Files 181 182 +1
Lines 12200 12249 +49
Branches 268 268
============================================
+ Hits 7551 7555 +4
- Misses 4061 4106 +45
Partials 588 588
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Signed-off-by: debasishbsws <debasishbsws.abc@gmail.com> Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
Signed-off-by: debasishbsws <debasishbsws.abc@gmail.com> Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
Signed-off-by: debasishbsws <debasishbsws.abc@gmail.com> Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
Signed-off-by: debasishbsws <debasishbsws.abc@gmail.com> Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
Signed-off-by: debasishbsws <debasishbsws.abc@gmail.com> Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
7c95306
to
31b59c6
Compare
/retest-required |
/test channel-integration-tests-ssl |
...main/java/dev/knative/eventing/kafka/broker/dispatcher/impl/filter/ExactFilterBenchmark.java
Outdated
Show resolved
Hide resolved
...src/main/java/dev/knative/eventing/kafka/broker/dispatcher/main/ConsumerVerticleBuilder.java
Show resolved
Hide resolved
Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
.../core/src/main/java/dev/knative/eventing/kafka/broker/core/tracing/kafka/ConsumerTracer.java
Outdated
Show resolved
Hide resolved
Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
/test build-tests |
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
/approve
Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
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
/approve
/retest
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: debasishbsws, matzew The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/retest-required |
1 similar comment
/retest-required |
/test channel-integration-tests-sasl-plain |
/retest |
/retest-required |
/test channel-integration-tests-ssl |
/retest |
1 similar comment
/retest |
21c92e7
into
knative-extensions:main
Progress #2928
Proposed Changes
final
ExactFilterBenchmark
andFilterBenchmark
via spotless. that might have been missed in the last PR Created abstract FilterBenchmark class #3247