Results 147 comments of Mike Miller

I was attempting to upgrade to 1.72 (closest to 1.71 that is in maven central) and had a similar issue. I tested multiple versions and narrowed it down to working...

I looked into the code that is discussed above and it would seem that it is behaving as intended. The side effect of these changes is that a ParameterException will...

> This might work, I remember that the cause was empty. Did you happen to test this to see what the message is for the parent exception? No, I wanted...

OK. It looks like the message is null too. Here is the full stacktrace: 2022-08-03T15:29:31,308 [fate.Fate] DEBUG: Failed to execute Repo, FATE[14ccaf316c860ecd] org.apache.accumulo.core.clientImpl.AcceptableThriftTableOperationException: null at org.apache.accumulo.manager.tableOps.Utils.reserveTable(Utils.java:96) ~[accumulo-manager-2.1.0-SNAPSHOT.jar:2.1.0-SNAPSHOT] at org.apache.accumulo.manager.tableOps.delete.DeleteTable.isReady(DeleteTable.java:45) ~[accumulo-manager-2.1.0-SNAPSHOT.jar:2.1.0-SNAPSHOT]...

OK I added the table ID, name and description to the log statement. They could also be null but if the information is there it will get printed and that...

> Reminder: please set the Project version(s) when merging PRs, so we can track the fix version for changes. FYI the original ticket had the version.

Thanks for taking a look at these methods. I will take a look at this today.

> > Depending on which PR is merged first, the properties changed here should match the design set in #2190. > > I'll finish reviewing #2190 so we can get...

I tested this branch on Uno locally using continuous ingest and was able to ingest 160 million entries in 20 minutes. The data verified correctly in about 15 minutes.

@ctubbsii mind taking a quick high level look at this branch before I redo the tests? I think I implemented the changes we discussed and got everything compiling (ignore tests...