datawave
datawave copied to clipboard
Replace usage of InMemoryInstance with InMemoryAccumulo
InMemoryInstance extends the Accumulo Instance class which was deprecated and removed in Accumulo 3. In the in-memory-accumulo/accumulo4 branch InMemoryInstance was removed because the upstream class went away. When attempting to compile datawave against accumulo 4 many changes were needed because InMemoryInstance no longer existed. These changes can be seen in the datawave accumulo4 branch. In the accumulo4 branch some of these usages were automatically changed using find and sed, see the commit message on 375793c51ea4651d611bbcdf56fb1bf8f5c5d274 for details.
If the integration branch were updated to remove use of InMemoryInstance this would reduce diffs between the integration and the accumulo4 branches. To do this a subset of the changes in https://github.com/NationalSecurityAgency/datawave-in-memory-accumulo/commit/67aa6d0330d328ff6c4b329807816b927c86828b may need to be pulled into the in-memory-accumulo main branch, like some of the constructor related changes to InMemoryAccumulo may be useful.