-
Notifications
You must be signed in to change notification settings - Fork 111
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
Fixing: NullPointerException
occurring and it's discarding the records while tracing for Loom deployment
#3300
Fixing: NullPointerException
occurring and it's discarding the records while tracing for Loom deployment
#3300
Conversation
Signed-off-by: debasishbsws <debasishbsws.dev@gmail.com>
NullPointerException
occurring and it's discarding the records while tracing. NullPointerException
occurring and it's discarding the records while tracing for Loom deployment
/cc @Cali0707 |
Codecov Report
@@ Coverage Diff @@
## main #3300 +/- ##
============================================
- Coverage 61.69% 61.68% -0.01%
- Complexity 767 768 +1
============================================
Files 182 182
Lines 12298 12304 +6
Branches 268 268
============================================
+ Hits 7587 7590 +3
- Misses 4118 4120 +2
- Partials 593 594 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
/home/prow/go/src/knative.dev/eventing-kafka-broker is out of date. Please run hack/update-codegen.sh
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 |
8c81822
into
knative-extensions:main
Problem
Current Loom Dispatcher was not working properly and there is no events retrieving on that side.
The Problem we find is that when we are using loom modules the ConsumerTracer (which is using vertx opentelemetry), is a
NullPointerException
occurring and it's discarding the records.Proposed Changes
As for the Solution we have here is we handle the records inside the current Vertx Context. So that when for the tracing we try to get the currentContext we get the context instead of
null
eventing-kafka-broker/data-plane/dispatcher/src/main/java/dev/knative/eventing/kafka/broker/dispatcher/impl/RecordDispatcherImpl.java
Lines 203 to 205 in c6e8f3c
/cc @pierDipi @matzew
Release Note