[
https://issues.apache.org/jira/browse/GOBBLIN-1563?focusedWorklogId=667196&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-667196
]
ASF GitHub Bot logged work on GOBBLIN-1563:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 19/Oct/21 18:53
Start Date: 19/Oct/21 18:53
Worklog Time Spent: 10m
Work Description: phet commented on a change in pull request #3416:
URL: https://github.com/apache/gobblin/pull/3416#discussion_r732155066
##########
File path:
gobblin-metrics-libs/gobblin-metrics-base/src/main/java/org/apache/gobblin/metrics/reporter/EventReporter.java
##########
@@ -144,9 +145,11 @@ public void notificationCallback(Notification
notification) {
*/
public void addEventToReportingQueue(GobblinTrackingEvent event) {
Review comment:
sorry, re-reading I see my reminder was poorly phrased.
just pointing out that derived classes, like `FFER`, only conditionally call
the method you added logging to. `KER` does not even override (so no current
issue)... merely reminding in general that non-`final` code may not actually
run, and to keep in mind, if things look off while later debugging.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 667196)
Time Spent: 2h 20m (was: 2h 10m)
> Collect more information to analyze the RC for some job cannot emit kafka
> events to update job status
> -----------------------------------------------------------------------------------------------------
>
> Key: GOBBLIN-1563
> URL: https://issues.apache.org/jira/browse/GOBBLIN-1563
> Project: Apache Gobblin
> Issue Type: Improvement
> Reporter: Zihan Li
> Priority: Major
> Time Spent: 2h 20m
> Remaining Estimate: 0h
>
> Add more log in Gobblin Metrics to collect info about why we see kafka events
> missing in some case
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
