You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@spark.apache.org by GitBox <gi...@apache.org> on 2019/02/04 19:16:26 UTC
[GitHub] MaxGekk commented on issue #23727: [SPARK-26817][CORE] Use
System.nanoTime to measure time intervals
MaxGekk commented on issue #23727: [SPARK-26817][CORE] Use System.nanoTime to measure time intervals
URL: https://github.com/apache/spark/pull/23727#issuecomment-460374697
I was trying to replace all places where `currentTimeMills` was used to calculate time deltas. I wasn't absolutely sure about some places and leave them as is. I will go over the places again and look at them more carefully. I skipped all places where start or end time can go outside, and can be used to track time lines.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@spark.apache.org
For additional commands, e-mail: reviews-help@spark.apache.org