I'll get to things soon

2016-09-21 Thread Jordan Zimmerman
FYI Sorry - I’m out of town for the next few weeks at a conference. I’ll try to get to Curator issues that Cameron hasn’t handled as soon as I can. -JZ

[jira] [Commented] (CURATOR-349) Expose extra metrics in TracerDriver

2016-09-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/CURATOR-349?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15511164#comment-15511164 ] ASF GitHub Bot commented on CURATOR-349: Github user cammckenzie commented on the

[GitHub] curator issue #165: [CURATOR-349] Expose extra metrics in TracerDriver

2016-09-21 Thread cammckenzie
Github user cammckenzie commented on the issue: https://github.com/apache/curator/pull/165 Thanks for the PR, The content looks OK, but I think that the interface changes need to be made backwards compatible. Can we change the OperationTrace to extend the existing TimerTrace?

[jira] [Commented] (CURATOR-350) Allow to close executor service on shutdown

2016-09-21 Thread ASF GitHub Bot (JIRA)
[ https://issues.apache.org/jira/browse/CURATOR-350?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15510298#comment-15510298 ] ASF GitHub Bot commented on CURATOR-350: Github user oliegeon closed the pull req

[GitHub] curator pull request #163: CURATOR-350 allow to close executor service on sh...

2016-09-21 Thread oliegeon
Github user oliegeon closed the pull request at: https://github.com/apache/curator/pull/163 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is e

[jira] [Closed] (CURATOR-350) Allow to close executor service on shutdown

2016-09-21 Thread Olivier Liegeon (JIRA)
[ https://issues.apache.org/jira/browse/CURATOR-350?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Olivier Liegeon closed CURATOR-350. --- Resolution: Invalid Thinking more about, it does not make any sense. Multiple watcher share t