SysInfoWindows gets the current time in nanonseconds but assumes milliseconds.
Move ResourceCalculatorPlugin from YARN to Common
When moving WindowsResourceCalculatorPlugin from YARN to Commons, the way to get the time changed. My proposal is to use Time.monotonicNow() which already does the translation from ns to ms.
First version of the patch to move ns into ms.
Chris Douglas, Karthik Kambatla as you guys were involved in HADOOP-12180, do you mind taking a look?
Not sure if there's a point on adding a unit test for this as it wouldn't run on Windows.
+1 lgtm. Agree this doesn't require a unit test; ran the TestSysInfo* tests, they passed on trunk/branch-2/branch-2.8.
Will commit when Jenkins comes back.
This message was automatically generated.
I committed this. Thanks Inigo
SUCCESS: Integrated in Hadoop-trunk-Commit #9705 (See https://builds.apache.org/job/Hadoop-trunk-Commit/9705/)
HADOOP-13080. Refresh time in SysInfoWindows is in nanoseconds. (cdouglas: rev c1cc6ac667e9e1b2ed58f16cb9fa1584ea54f0ac)
Thanks you Chris Douglas!