Branch: refs/heads/JENKINS-21251_take2
  Home:   https://github.com/jenkinsci/remoting
  Commit: 75fb3559e354418000760573eac19efcd86c267f
      
https://github.com/jenkinsci/remoting/commit/75fb3559e354418000760573eac19efcd86c267f
  Author: James Nord <jnord+git...@cloudbees.com>
  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 jenkinsci-commits+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to