[GitHub] [accumulo-testing] keith-turner commented on issue #111: AssertionError thrown where never should happen

2019-09-13 Thread GitBox
keith-turner commented on issue #111: AssertionError thrown where never should happen URL: https://github.com/apache/accumulo-testing/issues/111#issuecomment-531260936 I suspect in the following server side code the table was deleted after it resolved the table name to a table ID, but bef

[GitHub] [accumulo-testing] milleruntime commented on issue #111: AssertionError thrown where never should happen

2019-09-13 Thread GitBox
milleruntime commented on issue #111: AssertionError thrown where never should happen URL: https://github.com/apache/accumulo-testing/issues/111#issuecomment-531262227 I wasn't sure at the time so I opened it here. But I agree, this should be fixed in the 2.0 branch. I will open a PR.

[GitHub] [accumulo] keith-turner commented on issue #1358: Fix race conditions with commitsInProgress counter

2019-09-13 Thread GitBox
keith-turner commented on issue #1358: Fix race conditions with commitsInProgress counter URL: https://github.com/apache/accumulo/pull/1358#issuecomment-531274741 > For this particular PR it's still not clear to me why CommitSession.java needs to lock (block) the entire tablet to keep this

[GitHub] [accumulo] keith-turner edited a comment on issue #1358: Fix race conditions with commitsInProgress counter

2019-09-13 Thread GitBox
keith-turner edited a comment on issue #1358: Fix race conditions with commitsInProgress counter URL: https://github.com/apache/accumulo/pull/1358#issuecomment-531274741 > For this particular PR it's still not clear to me why CommitSession.java needs to lock (block) the entire tablet to ke

[GitHub] [accumulo] keith-turner edited a comment on issue #1358: Fix race conditions with commitsInProgress counter

2019-09-13 Thread GitBox
keith-turner edited a comment on issue #1358: Fix race conditions with commitsInProgress counter URL: https://github.com/apache/accumulo/pull/1358#issuecomment-531274741 > For this particular PR it's still not clear to me why CommitSession.java needs to lock (block) the entire tablet to ke

[GitHub] [accumulo] milleruntime commented on a change in pull request #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
milleruntime commented on a change in pull request #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#discussion_r324265874 ## File path: server/gc/src/main/java/org/apache/accumulo/gc/SimpleGarbageCollector.java ## @@ -86

[GitHub] [accumulo] milleruntime commented on a change in pull request #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
milleruntime commented on a change in pull request #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#discussion_r324265874 ## File path: server/gc/src/main/java/org/apache/accumulo/gc/SimpleGarbageCollector.java ## @@ -86

[GitHub] [accumulo] milleruntime opened a new issue #1361: Monitor not displaying dead Tservers

2019-09-13 Thread GitBox
milleruntime opened a new issue #1361: Monitor not displaying dead Tservers URL: https://github.com/apache/accumulo/issues/1361 I killed a tserver and it is not showing up in the dead tservers table. ![monitor-dead-tserver2](https://user-images.githubusercontent.com/11872539/64887976-d49

[GitHub] [accumulo] jzgithub1 commented on issue #1225: Use fewer ZooKeeper watchers

2019-09-13 Thread GitBox
jzgithub1 commented on issue #1225: Use fewer ZooKeeper watchers URL: https://github.com/apache/accumulo/issues/1225#issuecomment-531360046 @ctubbsii. I investigated how I would implement:" _One possible solution is instead of having a watcher for every configuration item, we have only a s

[GitHub] [accumulo] milleruntime opened a new issue #1362: Table Status page is broken

2019-09-13 Thread GitBox
milleruntime opened a new issue #1362: Table Status page is broken URL: https://github.com/apache/accumulo/issues/1362 In the Monitor when on the Tables page... click a table ID link and it should refresh the page with detailed information about that table. I am getting this JS error and a

[GitHub] [accumulo] keith-turner commented on issue #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
keith-turner commented on issue #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#issuecomment-531365357 @EdColeman are you only interested in the last stats or the last and current stats. Just getting the last stats could simple because

[GitHub] [accumulo] milleruntime commented on issue #1225: Use fewer ZooKeeper watchers

2019-09-13 Thread GitBox
milleruntime commented on issue #1225: Use fewer ZooKeeper watchers URL: https://github.com/apache/accumulo/issues/1225#issuecomment-531372663 @jzgithub1 I recommend putting long output like this in a text file and attaching it to your comment by dragging and dropping into the comment box.

[GitHub] [accumulo] milleruntime commented on issue #471: Retry when conncurrent merge happens in bulk import.

2019-09-13 Thread GitBox
milleruntime commented on issue #471: Retry when conncurrent merge happens in bulk import. URL: https://github.com/apache/accumulo/issues/471#issuecomment-531380062 I just saw this running random walk concurrent test on 2.0. ``` java.lang.Exception: Error running node Concurrent.xml

[GitHub] [accumulo] milleruntime commented on issue #1362: Table Status page is broken

2019-09-13 Thread GitBox
milleruntime commented on issue #1362: Table Status page is broken URL: https://github.com/apache/accumulo/issues/1362#issuecomment-531390457 This was an issue with my proxy not refreshing. A good old "Ctrl + F5" fixed the issue. ---

[GitHub] [accumulo] milleruntime closed issue #1362: Table Status page is broken

2019-09-13 Thread GitBox
milleruntime closed issue #1362: Table Status page is broken URL: https://github.com/apache/accumulo/issues/1362 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [accumulo] milleruntime commented on issue #1361: Monitor not displaying dead Tservers

2019-09-13 Thread GitBox
milleruntime commented on issue #1361: Monitor not displaying dead Tservers URL: https://github.com/apache/accumulo/issues/1361#issuecomment-531392362 Not an issue. Clearing browser cache fixed this. This is an automated mess

[GitHub] [accumulo] milleruntime closed issue #1361: Monitor not displaying dead Tservers

2019-09-13 Thread GitBox
milleruntime closed issue #1361: Monitor not displaying dead Tservers URL: https://github.com/apache/accumulo/issues/1361 This is an automated message from the Apache Git Service. To respond to the message, please log on to G

[GitHub] [accumulo] EdColeman commented on a change in pull request #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
EdColeman commented on a change in pull request #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#discussion_r324403386 ## File path: server/gc/src/main/java/org/apache/accumulo/gc/SimpleGarbageCollector.java ## @@ -865,6

[GitHub] [accumulo] EdColeman commented on issue #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
EdColeman commented on issue #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#issuecomment-531428566 @keith-turner I'll post up an approach that uses local, thread safe stats and then converts to the thrift values when requested and we can

[GitHub] [accumulo] EdColeman edited a comment on issue #1360: Improve concurrent access to gc statistics.

2019-09-13 Thread GitBox
EdColeman edited a comment on issue #1360: Improve concurrent access to gc statistics. URL: https://github.com/apache/accumulo/pull/1360#issuecomment-531428566 I'm interested in being able to publish the last stats via hadoop metrics. @keith-turner I'll post up an approach that uses