Issue1421

[phrocker]

[phrocker]

@keith-ratcliffe I wasn't able to build this locally to be able to run the formatter ( I submitted from windows, hence the line break changes, which should go away when I can successfully build on linux ). Note that I initially created this and built this in a separate branch ( branched from 2.0 I believe ), where tests passed; however, I copied these changes into a branch from feature/accumulo-2.1 where I had some conflicts I resolved through github but wasn't able to build and validate locally. Any guidance would help so I can make sure this branch works

[tomnelson]

I don't think that the automated tests will work until the accumulo-2.1-SNAPSHOT jars are in an available repo, and until the datawave-in-memory-accumulo 'feature/accumulo-2.1' branch is updated to fix breaking changes in accumulo-2.1-SNAPSHOT, and until the datawave-in-memory-accumulo 2.1 snapshot jars are also in some repo somewhere.

[keith-ratcliffe]

@phrocker sorry for the hassle. There's still quite a bit of churn on the upstream Accumulo 2.1.x, which is why I haven't released the dependencies you were lacking in your build. For now, if you clone these...

https://github.com/NationalSecurityAgency/datawave-in-memory-accumulo https://github.com/NationalSecurityAgency/datawave-metadata-utils https://github.com/NationalSecurityAgency/datawave-accumulo-utils

...and mvn clean install the feature/accumulo-2.1 branch on each repo, you should be able to build the entire project.

Note that you'll probably also need to clone/build the latest Accumulo 2.1.0-SNAPSHOT as a prerequisite, as I just merged @tomnelson's datawave-in-memory-accumulo PR, which fixes breaking changes stemming from very recent Accumulo changes.

[phrocker]

Thanks @keith-ratcliffe . I had to make similar changes; however, I still have some build failures in webservice. I'll investigate those.