GitHub user wilderrodrigues opened a pull request:
https://github.com/apache/cloudstack/pull/509
Replace System.currentTimeMillis() by System.nanoTime()
- System.nanoTime() is the best way to measure elapsed time in Java.
- It gives a resolution on the order of microseconds
- The System.currentTimeMillis() is used when calculating absolut time.
Add unit tests to cover negative cases
- Cover when the profile is not started/stopped
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/schubergphilis/cloudstack
improvement/system_nanotime
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/509.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #509
----
commit 303adb3117b3039aa4b0dbc91540627d06bcb07c
Author: wilderrodrigues <wrodrig...@schubergphilis.com>
Date: 2015-06-22T12:07:33Z
Replace System.currentTimeMillis() by System.nanoTime()
- System.nanoTime() is the best way to measure elapsed time in Java.
- It gives a resolution on the order of microseconds
The System.currentTimeMillis() is used when calculating absolut time.
commit b8d46ff4f78f5996c68d86477ec894fea636e866
Author: wilderrodrigues <wrodrig...@schubergphilis.com>
Date: 2015-06-22T14:05:15Z
Add unit tests to cover negative cases
- Cover when the profile is not started/stopped
----
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
