Github user unsleepy22 commented on the pull request:
https://github.com/apache/storm/pull/1143#issuecomment-188099708
I agree with @darionyaphet . Actually I think all System.nanoTime callings
in external modules should be replaced with System.currentTimeMillis since they
use System.nanoTime to get a ns value and then convert the time back to ms,
which is unnecessary.
System.nanoTime should only be used in such cases that we do need high
precision timing, e.g., timing certain code snippets in metrics system. If
called on a critical path, this method is 5+ times slower than
System.currentTimeMillis.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
