vivek807 commented on PR #19603: URL: https://github.com/apache/druid/pull/19603#issuecomment-4796964688
> @vivek807 Thanks for flagging this issue. You are right, the values are not comparable. I'm thinking that maybe instead we should standardize the timestamps in OnDemandDatabasePoll and PeriodicDatabasePoll to use the same format (i.e. both use System.nanoTime() or both use System.currentTimeMillis()). This can avoid similar confusion / bug like this in the future. Wdyt? Thanks! Yes, this will avoid confusion in future. Let me refactor it to use same type of clock here -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
