[GitHub] [accumulo] arvindshmicrosoft commented on issue #1464: Leverage SimpleThreadPool for improving importtable performance

arvindshmicrosoft commented on issue #1464: Leverage SimpleThreadPool for improving importtable performance URL: https://github.com/apache/accumulo/issues/1464#issuecomment-585007111 FYI I'm starting to work on this, so if this issue can be assigned to me it would be appreciated.

[GitHub] [accumulo] milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements

milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements URL: https://github.com/apache/accumulo/issues/1502#issuecomment-584631321 I was able to confirm that "something is not right" when taking your suggestion on #1501 and

[GitHub] [accumulo] milleruntime merged pull request #1501: Use TabletFile for Tablet, Scans and compaction

milleruntime merged pull request #1501: Use TabletFile for Tablet, Scans and compaction URL: https://github.com/apache/accumulo/pull/1501 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [accumulo] milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements

milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements URL: https://github.com/apache/accumulo/issues/1502#issuecomment-584634187 Some more info from VolumeIT.testCleanReplaceVolumes() hanging... a jstack on the pid shows:

[GitHub] [accumulo] milleruntime edited a comment on issue #1502: Determine if files are deleted from metadata table when using volume.replacements

milleruntime edited a comment on issue #1502: Determine if files are deleted from metadata table when using volume.replacements URL: https://github.com/apache/accumulo/issues/1502#issuecomment-584668939 Nevermind this stacktrace was for a bug I had in my code.

[GitHub] [accumulo] milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements

milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements URL: https://github.com/apache/accumulo/issues/1502#issuecomment-584668939 > Some more info from VolumeIT.testCleanReplaceVolumes() hanging... a jstack on the pid

Accumulo-Master - Build # 3017 - Fixed

The Apache Jenkins build system has built Accumulo-Master (build #3017) Status: Fixed Check console output at https://builds.apache.org/job/Accumulo-Master/3017/ to view the results.

[GitHub] [accumulo] karthick-rn commented on issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5

karthick-rn commented on issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5 URL: https://github.com/apache/accumulo/issues/1503#issuecomment-584700257 @billierinaldi, Can I start working on this? This is an automated

[GitHub] [accumulo] ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout

ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584709467 @MuazmaZ Were you able to run the ITs with this timeout? Did you notice any particularly problematic tests that might need

[GitHub] [accumulo] billierinaldi commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout

billierinaldi commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584725797 I'm attempting mvn verify with this change on my laptop now (with a timeout.factor). But I haven't run a verify on this

[GitHub] [accumulo] billierinaldi commented on issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5

billierinaldi commented on issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5 URL: https://github.com/apache/accumulo/issues/1503#issuecomment-584702996 Sure! Thanks, @karthick-rn. This is an automated message from

[GitHub] [accumulo] MuazmaZ opened a new pull request #1504: Updated AccumuloITBase.java to enforce a default timeout

MuazmaZ opened a new pull request #1504: Updated AccumuloITBase.java to enforce a default timeout URL: https://github.com/apache/accumulo/pull/1504 Updated default TimeoutSeconds() method to be 10 minutes This is an

[GitHub] [accumulo-website] ctubbsii merged pull request #220: Make git instructions slightly shorter

ctubbsii merged pull request #220: Make git instructions slightly shorter URL: https://github.com/apache/accumulo-website/pull/220 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [accumulo] ctubbsii commented on issue #1486: Broken SplitRecoveryIT

ctubbsii commented on issue #1486: Broken SplitRecoveryIT URL: https://github.com/apache/accumulo/issues/1486#issuecomment-584742757 @milleruntime If you're confident that fixed it, I'll go ahead and close this issue. We can reopen if we see it again.

[GitHub] [accumulo] ctubbsii closed issue #1486: Broken SplitRecoveryIT

ctubbsii closed issue #1486: Broken SplitRecoveryIT URL: https://github.com/apache/accumulo/issues/1486 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [accumulo] milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements

milleruntime commented on issue #1502: Determine if files are deleted from metadata table when using volume.replacements URL: https://github.com/apache/accumulo/issues/1502#issuecomment-584685220 I was able to see an error when initializing a Tablet with the replaced volumes but this is

[GitHub] [accumulo] MuazmaZ commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout

MuazmaZ commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584719269 @ctubbsii I didn't see any test, in particular, to be problematic

[GitHub] [accumulo] ctubbsii commented on issue #1488: ManyWriteAheadLogsIT is flakey

ctubbsii commented on issue #1488: ManyWriteAheadLogsIT is flakey URL: https://github.com/apache/accumulo/issues/1488#issuecomment-584741511 @Manno15 I was running the entire suite in Jenkins, but to reproduce, I would probably repeatedly run something like: `mvn clean verify -Dtest=blah

[GitHub] [accumulo] Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey

Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey URL: https://github.com/apache/accumulo/issues/1488#issuecomment-584740705 I am having trouble reproducing this error, both on 1.9 and on the master branch. What command did you use to run this IT or did you run the entire

[GitHub] [accumulo] billierinaldi opened a new issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5

billierinaldi opened a new issue #1503: Make branch 1.9 compatible with ZooKeeper 3.5 URL: https://github.com/apache/accumulo/issues/1503 In ZooKeeper 3.5, the zookeeper jar has been moved from ZOOKEEPER_HOME to ZOOKEEPER_HOME/lib. Some things that need to be fixed in the 1.9 branch to

[GitHub] [accumulo] ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout

ctubbsii commented on issue #1504: Updated AccumuloITBase.java to enforce a default timeout URL: https://github.com/apache/accumulo/pull/1504#issuecomment-584740191 I would be happy to merge this in now, but I'm seeing other failures currently on the master branch, and would like to see

[GitHub] [accumulo] Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey

Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey URL: https://github.com/apache/accumulo/issues/1488#issuecomment-584745190 @ctubbsii That's how I am running it. Still not hitting the error. Interesting. This

[GitHub] [accumulo] milleruntime opened a new pull request #1505: Optimize Tablet volume replacement

milleruntime opened a new pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505 * Stop Tablet constructor from checking every file for volume replacements when feature not used

[GitHub] [accumulo] ctubbsii commented on issue #1488: ManyWriteAheadLogsIT is flakey

ctubbsii commented on issue #1488: ManyWriteAheadLogsIT is flakey URL: https://github.com/apache/accumulo/issues/1488#issuecomment-584758069 It might be rare. We could close it as not reproducible, but I'd feel a bit more comfortable if we had an explanation for how the above error could

Accumulo-Master - Build # 3018 - Aborted

The Apache Jenkins build system has built Accumulo-Master (build #3018) Status: Aborted Check console output at https://builds.apache.org/job/Accumulo-Master/3018/ to view the results.

[GitHub] [accumulo] EdColeman commented on a change in pull request #1505: Optimize Tablet volume replacement

EdColeman commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377831812 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8 @@

[GitHub] [accumulo] ivakegg commented on a change in pull request #1505: Optimize Tablet volume replacement

ivakegg commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377831737 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8 @@

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1505: Optimize Tablet volume replacement

ctubbsii commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377831890 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8 @@

[GitHub] [accumulo] Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey

Manno15 commented on issue #1488: ManyWriteAheadLogsIT is flakey URL: https://github.com/apache/accumulo/issues/1488#issuecomment-584759217 I will take a deeper investigation into the IT. This is an automated message from

[GitHub] [accumulo] ctubbsii commented on issue #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis

ctubbsii commented on issue #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis URL: https://github.com/apache/accumulo/pull/1489#issuecomment-584789584 I think the failures you're talking about with the ITs can happen with or without this change, and would consider

[GitHub] [accumulo] milleruntime commented on a change in pull request #1505: Optimize Tablet volume replacement

milleruntime commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377830031 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8

[GitHub] [accumulo] milleruntime commented on a change in pull request #1505: Optimize Tablet volume replacement

milleruntime commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377834567 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8

[GitHub] [accumulo] ctubbsii commented on a change in pull request #1505: Optimize Tablet volume replacement

ctubbsii commented on a change in pull request #1505: Optimize Tablet volume replacement URL: https://github.com/apache/accumulo/pull/1505#discussion_r377803422 ## File path: server/base/src/main/java/org/apache/accumulo/server/fs/VolumeUtil.java ## @@ -184,9 +184,8 @@

[GitHub] [accumulo] ctubbsii commented on issue #1430: Limit the size of new table names

ctubbsii commented on issue #1430: Limit the size of new table names URL: https://github.com/apache/accumulo/issues/1430#issuecomment-584771958 Awesome @cradal . Let me know if you run into any issues. This is an automated

[GitHub] [accumulo] ctubbsii closed issue #1458: ZooSession should use nanoTime instead of currentTimeMillis

ctubbsii closed issue #1458: ZooSession should use nanoTime instead of currentTimeMillis URL: https://github.com/apache/accumulo/issues/1458 This is an automated message from the Apache Git Service. To respond to the

[GitHub] [accumulo] ctubbsii merged pull request #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis

ctubbsii merged pull request #1489: Fixes #1458 - ZooSession now uses nanoTime vice currentTimeMillis URL: https://github.com/apache/accumulo/pull/1489 This is an automated message from the Apache Git Service. To respond to

[GitHub] [accumulo] jzgithub1 commented on issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails)

jzgithub1 commented on issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails) URL: https://github.com/apache/accumulo/issues/1435#issuecomment-584838036 @ctubbsii , this is the only error I see when running this test but it is not related to OOME. The call to deleteLock in

[GitHub] [accumulo] karthick-rn opened a new pull request #1506: Make Accumulo 1.9 compatible with ZK 3.5

karthick-rn opened a new pull request #1506: Make Accumulo 1.9 compatible with ZK 3.5 URL: https://github.com/apache/accumulo/pull/1506 Fix as described in #1503 This is an automated message from the Apache Git Service. To

[GitHub] [accumulo] karthick-rn commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5

karthick-rn commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5 URL: https://github.com/apache/accumulo/pull/1506#issuecomment-584874242 Thanks @ctubbsii. Corrected the branch. This is an automated message from

[GitHub] [accumulo] billierinaldi commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5

billierinaldi commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5 URL: https://github.com/apache/accumulo/pull/1506#issuecomment-584876874 Thanks for the patch, @karthick-rn! For the general.classpaths change, we should keep both ZK jar locations on the classpath so that it

[GitHub] [accumulo] keith-turner commented on issue #1267: Research if Ample should have sanity checks for no tablets seen

keith-turner commented on issue #1267: Research if Ample should have sanity checks for no tablets seen URL: https://github.com/apache/accumulo/issues/1267#issuecomment-584883327 @milleruntime I think this is more similar to #1377. I was thinking of the case where the master and gc scan

[GitHub] [accumulo] keith-turner closed issue #1267: Research if Ample should have sanity checks for no tablets seen

keith-turner closed issue #1267: Research if Ample should have sanity checks for no tablets seen URL: https://github.com/apache/accumulo/issues/1267 This is an automated message from the Apache Git Service. To respond to

[GitHub] [accumulo] ctubbsii commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5

ctubbsii commented on issue #1506: Make Accumulo 1.9 compatible with ZK 3.5 URL: https://github.com/apache/accumulo/pull/1506#issuecomment-584862942 The title indicates it is for 1.9, but the PR was made against the master branch.

[GitHub] [accumulo] ctubbsii commented on issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails)

ctubbsii commented on issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails) URL: https://github.com/apache/accumulo/issues/1435#issuecomment-584864715 I think this was fixed in #1441 (b0bbff007d372fb3a9e3c4c4f900e063a65a40b1)

[GitHub] [accumulo] ctubbsii closed issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails)

ctubbsii closed issue #1435: Master OOME during GCUpgrade9to10TestIT (test fails) URL: https://github.com/apache/accumulo/issues/1435 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [accumulo] milleruntime opened a new issue #1507: Throw Exception to prevent incorrect metadata

milleruntime opened a new issue #1507: Throw Exception to prevent incorrect metadata URL: https://github.com/apache/accumulo/issues/1507 Follow on work for #1501. With the new TabletFile, we can now prevent metadata paths being incorrectly written (or not deleted) if an exception is

[GitHub] [accumulo] milleruntime commented on issue #1267: Research if Ample should have sanity checks for no tablets seen

milleruntime commented on issue #1267: Research if Ample should have sanity checks for no tablets seen URL: https://github.com/apache/accumulo/issues/1267#issuecomment-584879313 @keith-turner is this the same as https://github.com/apache/accumulo/issues/1507 ? It sounds similar...