[GitHub] accumulo pull request #150: ACCUMULO-4424 Start thrift servers immediately o...

2016-09-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/accumulo/pull/150 --- 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 en

[GitHub] accumulo issue #150: ACCUMULO-4424 Start thrift servers immediately on ha co...

2016-09-12 Thread ctubbsii
Github user ctubbsii commented on the issue: https://github.com/apache/accumulo/pull/150 +1 to the strategy. Haven't checked out the code to test. The test case looks good, though. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub

[GitHub] accumulo issue #150: ACCUMULO-4424 Start thrift servers immediately on ha co...

2016-09-12 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/accumulo/pull/150 b36d9a8 sends HTTP/503 with the error message instead of propagating the exception. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well

[GitHub] accumulo pull request #150: ACCUMULO-4424 Start thrift servers immediately o...

2016-09-12 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/150#discussion_r78467505 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/BasicServlet.java --- @@ -51,8 +52,18 @@ abstract protected Stri

[GitHub] accumulo pull request #150: ACCUMULO-4424 Start thrift servers immediately o...

2016-09-12 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/150#discussion_r78467243 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/BasicServlet.java --- @@ -51,8 +52,18 @@ abstract protected Strin

[GitHub] accumulo issue #150: ACCUMULO-4424 Start thrift servers immediately on ha co...

2016-09-12 Thread joshelser
Github user joshelser commented on the issue: https://github.com/apache/accumulo/pull/150 Just pushed an IT for this change in 2f060a4. --- 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 f

[GitHub] accumulo pull request #150: ACCUMULO-4424 Start thrift servers immediately o...

2016-09-12 Thread joshelser
Github user joshelser commented on a diff in the pull request: https://github.com/apache/accumulo/pull/150#discussion_r78464250 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/BasicServlet.java --- @@ -51,8 +52,18 @@ abstract protected Stri

[GitHub] accumulo pull request #150: ACCUMULO-4424 Start thrift servers immediately o...

2016-09-12 Thread ctubbsii
Github user ctubbsii commented on a diff in the pull request: https://github.com/apache/accumulo/pull/150#discussion_r78462981 --- Diff: server/monitor/src/main/java/org/apache/accumulo/monitor/servlets/BasicServlet.java --- @@ -51,8 +52,18 @@ abstract protected Strin