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.