Branch: refs/heads/JENKINS-21251_take2
Home: https://github.com/jenkinsci/remoting
Commit: 75fb3559e354418000760573eac19efcd86c267f
https://github.com/jenkinsci/remoting/commit/75fb3559e354418000760573eac19efcd86c267f
Author: James Nord <[email protected]>
Date: 2015-02-20 (Fri, 20 Feb 2015)
Changed paths:
M src/main/java/hudson/remoting/AtmostOneThreadExecutor.java
M src/main/java/hudson/remoting/Channel.java
M src/main/java/hudson/remoting/PingThread.java
M src/main/java/hudson/remoting/Request.java
M src/main/java/hudson/remoting/SingleLaneExecutorService.java
M src/main/java/hudson/remoting/SynchronousExecutorService.java
Log Message:
-----------
[JENKINS-21251] Initial calculation of wait time was susseptable to
clock drift.
The calculation on how long to wait in various places on the code was
suseptable to the clock changing as multiple calls where made to
System.currentTimeMillis() for the initial calculation.
Changed this so that we use a single call to System.nanoTime and made just
a single call for the initial calculation.
There is still a potential issue for any callers of Channel.getLastHeard()
this will be addressed in a future change and has only been noted in this
commit.
--
You received this message because you are subscribed to the Google Groups
"Jenkins Commits" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
