Branch: refs/heads/master
  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.


  Commit: c6ec2305614deab8ab08b3dd7262a909fba1e5bf
      
https://github.com/jenkinsci/remoting/commit/c6ec2305614deab8ab08b3dd7262a909fba1e5bf
  Author: James Nord <jnord+git...@cloudbees.com>
  Date:   2015-02-20 (Fri, 20 Feb 2015)

  Changed paths:
    M src/main/java/hudson/remoting/SynchronousExecutorService.java

  Log Message:
  -----------
  prevent infinite duration (0) in wait


  Commit: 0ab640376059daa0b0b04e8acf4619accb083837
      
https://github.com/jenkinsci/remoting/commit/0ab640376059daa0b0b04e8acf4619accb083837
  Author: James Nord <te...@teilo.net>
  Date:   2015-03-03 (Tue, 03 Mar 2015)

  Changed paths:
    M src/main/java/hudson/remoting/PingThread.java

  Log Message:
  -----------
  Update PingThread.java

In the loop set the minimum sleep duration to 1 ns.


  Commit: 7f35e250f5d535a9dbe16e67a47443aab568e7c4
      
https://github.com/jenkinsci/remoting/commit/7f35e250f5d535a9dbe16e67a47443aab568e7c4
  Author: James Nord <te...@teilo.net>
  Date:   2015-03-13 (Fri, 13 Mar 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:
  -----------
  Merge pull request #33 from jenkinsci/JENKINS-21251_take2

[JENKINS-21251] Slave channel timeout seemingly caused by usage of 
System.currentTimeMillis()


Compare: 
https://github.com/jenkinsci/remoting/compare/c4df5cb9c458...7f35e250f5d5

-- 
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