umustafi commented on PR #3788: URL: https://github.com/apache/gobblin/pull/3788#issuecomment-1734578922
> nice catch! I was wondering if it even makes sense for us to provide `eventTimeMillis` to the method signature of `scheduleReminderForEvent`. Earlier we were using it to create `JobDetail`, but now the only place we use it is for logging and the info that it conveys is the clock drift (which we might be okay not logging)... let me know what's your take? I am leaving it in to catch that clock drift in time being. I suspect it's not the root of our problem but until we are reliably bug free keeping logs in. -- 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]
