You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2020/02/03 13:38:11 UTC

[GitHub] [accumulo] cradal commented on issue #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis

cradal commented on issue #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis
URL: https://github.com/apache/accumulo/pull/1489#issuecomment-581417459
 
 
   I incorporated your suggestion to calculate a duration value and then convert to milliseconds. Just to be overly thorough, I compared the previous equations with the updated ones to make sure they are mathematically equivalent. This compiles fine. However, on about half of the Sunnyday test runs, the 2x zimeout period RuntimeException is thrown. Since this is supposed to happen if duration > 2X timeout, should we be concerned if that is the only test failure?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services