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 2022/02/23 19:51:37 UTC
[GitHub] [accumulo] DomGarguilo opened a new pull request #2521: Change timeouts to correct unit in accumulo-core module
DomGarguilo opened a new pull request #2521:
URL: https://github.com/apache/accumulo/pull/2521
In #2427 I converted to the new syntax for timeouts but overlooked the fact that the default units have changed from milliseconds to seconds. This PR changes the timeouts back to their intended duration.
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] DomGarguilo commented on pull request #2521: Change timeouts to correct unit in accumulo-core module
Posted by GitBox <gi...@apache.org>.
DomGarguilo commented on pull request #2521:
URL: https://github.com/apache/accumulo/pull/2521#issuecomment-1049156544
This relates to [my comment here](https://github.com/apache/accumulo/issues/2441#issuecomment-1049133153). Depending on what we decide, some of these could be changed to include units.
For example:
`@Timeout(60)` would become `@Timeout(value = 1, unit = MINUTES)`
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] ctubbsii merged pull request #2521: Change timeouts to correct unit in accumulo-core module
Posted by GitBox <gi...@apache.org>.
ctubbsii merged pull request #2521:
URL: https://github.com/apache/accumulo/pull/2521
--
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.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org