[GitHub] [accumulo] Manno15 commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747381390 The test with agitation also went well. 7 server, technically 8 node cluster. Received the same warnings as above and the only other warnings/errors are I think due to the

[GitHub] [accumulo] Manno15 edited a comment on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 edited a comment on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747381390 The test with agitation also went well. 7 server, technically 8 node cluster. Received the same warnings as above and the only other warnings/errors are I think due to

[GitHub] [accumulo] Manno15 commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747570194 This is what I am running into with trying to verify using accumulo-testing cingest/verify. I understand the error but nothing is working to fix it. ``` Exception in

[GitHub] [accumulo] dlmarion commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
dlmarion commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747575246 So you are running with Java 8 and the jar file was compile with Java 11, for example. That jar will need to be re-compiled with Java 8.

[GitHub] [accumulo] Manno15 commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747579226 I've tried that but I will try again. This is an automated message from the Apache Git Service. To respond to

[GitHub] [accumulo] dlmarion commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
dlmarion commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747567220 Do you have a stack trace? This is an automated message from the Apache Git Service. To respond to the

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
ctubbsii commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545389383 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }

[GitHub] [accumulo] EdColeman commented on issue #637: Ensure ZK server failure in the middle of lock creation is handled

2020-12-17 Thread GitBox
EdColeman commented on issue #637: URL: https://github.com/apache/accumulo/issues/637#issuecomment-747688533 This "type: of issue was also discussed in https://github.com/apache/accumulo/issues/1086. This is an automated

[GitHub] [accumulo] EdColeman edited a comment on issue #637: Ensure ZK server failure in the middle of lock creation is handled

2020-12-17 Thread GitBox
EdColeman edited a comment on issue #637: URL: https://github.com/apache/accumulo/issues/637#issuecomment-747688533 This "type" of issue was also discussed in https://github.com/apache/accumulo/issues/1086. This is an

[GitHub] [accumulo] Manno15 commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747638788 I got it to work. I forgot hadoop has an export JAVA_HOME of its own and that was exporting java 8 still for some reason. Thanks for your help guys. I will update you with the

[GitHub] [accumulo] DomGarguilo commented on issue #637: Ensure ZK server failure in the middle of lock creation is handled

2020-12-17 Thread GitBox
DomGarguilo commented on issue #637: URL: https://github.com/apache/accumulo/issues/637#issuecomment-747671402 The attached link suggests the use of `.withProtection()` while creating the ephemeral sequential nodes. From what I can tell, this is an apache curator function which is not

[GitHub] [accumulo] ctubbsii opened a new issue #1841: Broken or Flaky test: ConcurrentDeleteTableIT takes 30 minutes to run!!!

2020-12-17 Thread GitBox
ctubbsii opened a new issue #1841: URL: https://github.com/apache/accumulo/issues/1841 **Test name(s)** - ConcurrentDeleteTableIT.testConcurrentDeleteTablesOps took 587 seconds (nearly 10 minutes) - ConcurrentDeleteTableIT.testConcurrentFateOpsWithDelete took 1,152 seconds (nearly

[GitHub] [accumulo] ctubbsii commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
ctubbsii commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747603412 Accumulo 2.1.0-SNAPSHOT actually requires Java 11. You shouldn't need to recompile with Java 8 (accumulo-testing already compiles as Java 8 using the `-release 8` flag). You

[GitHub] [accumulo] Manno15 commented on pull request #1818: Fixes #1808 - Stop server side VM on Error in Thread

2020-12-17 Thread GitBox
Manno15 commented on pull request #1818: URL: https://github.com/apache/accumulo/pull/1818#issuecomment-747616460 I did and still get the same error. I tried on a different node too and nothing. It does work locally so must be something related to the cluster.

[GitHub] [accumulo] dlmarion commented on issue #1606: Ability to disable scan file limit in FileManager

2020-12-17 Thread GitBox
dlmarion commented on issue #1606: URL: https://github.com/apache/accumulo/issues/1606#issuecomment-747650183 @FineAndDandy - any updated information on the question in the comment above? Is this still an issue? This is an

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545391850 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/LookupTask.java ## @@ -177,9 +177,13 @@ public void run() { }

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545392485 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
ctubbsii commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545345733 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/LookupTask.java ## @@ -177,9 +177,13 @@ public void run() { }

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545362727 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
ctubbsii commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545391316 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +89,15 @@ public void run() { }

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545364298 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545364298 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }

[GitHub] [accumulo] ctubbsii commented on issue #637: Ensure ZK server failure in the middle of lock creation is handled

2020-12-17 Thread GitBox
ctubbsii commented on issue #637: URL: https://github.com/apache/accumulo/issues/637#issuecomment-747686478 @DomGarguilo I think the idea was to see how Curator solves the issue, to see if we can adopt a similar strategy in the relevant code in Accumulo. We have previously attempted (a

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
ctubbsii commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545388080 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/LookupTask.java ## @@ -177,9 +177,13 @@ public void run() { }

[GitHub] [accumulo] ctubbsii opened a new pull request #1842: Improve some IT framework and build-related code

2020-12-17 Thread GitBox
ctubbsii opened a new pull request #1842: URL: https://github.com/apache/accumulo/pull/1842 * Increase the default memory for mini-based ITs (from 128MB to 256MB) The justification for this is that using the G1GC in newer JVMs seems to use a little bit more base memory than the

[GitHub] [accumulo] dlmarion opened a new pull request #1840: Closes #1689: Don't catch Throwable (unless its rethrown)

2020-12-17 Thread GitBox
dlmarion opened a new pull request #1840: URL: https://github.com/apache/accumulo/pull/1840 Modified code that was catching Throwable to mostly catch Exception, unless Throwable was being re-thrown. This, in conjunction with https://github.com/apache/accumulo/pull/1818, should expose

[GitHub] [accumulo] dlmarion commented on issue #1559: Consider implementing failsafe to prevent large number of tablet assignments in bulk import

2020-12-17 Thread GitBox
dlmarion commented on issue #1559: URL: https://github.com/apache/accumulo/issues/1559#issuecomment-747652022 @milleruntime - should this issue be closed as the PR for it is merged? Is there something still left to do? This

[GitHub] [accumulo] ctubbsii commented on issue #1559: Consider implementing failsafe to prevent large number of tablet assignments in bulk import

2020-12-17 Thread GitBox
ctubbsii commented on issue #1559: URL: https://github.com/apache/accumulo/issues/1559#issuecomment-747659563 @dlmarion I think this was just overlooked and not automatically closed because the original linked PR was for 1.10, and the 2.1 PR that was merged was separate. I can close it

[GitHub] [accumulo] ctubbsii closed issue #1559: Consider implementing failsafe to prevent large number of tablet assignments in bulk import

2020-12-17 Thread GitBox
ctubbsii closed issue #1559: URL: https://github.com/apache/accumulo/issues/1559 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

[GitHub] [accumulo] dlmarion commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
dlmarion commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545361484 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/LookupTask.java ## @@ -177,9 +177,13 @@ public void run() { }

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1840: Closes #1689: Don't catch Throwable (unless it's rethrown)

2020-12-17 Thread GitBox
ctubbsii commented on a change in pull request #1840: URL: https://github.com/apache/accumulo/pull/1840#discussion_r545405013 ## File path: server/tserver/src/main/java/org/apache/accumulo/tserver/scan/NextBatchTask.java ## @@ -89,14 +88,16 @@ public void run() { }