Moriarty
Moriarty
> Added new "coverage" module and made changes to enable the SonarQube/Jacoco code coverage reports. Uses a local SonarQube instance to keep track of the coverage report. It can run...
Alternatively, https://github.com/NationalSecurityAgency/datawave/pull/2253
Closing due to inactivity
> Creating for @apmoriarty . Unsure if it's of value since I'm focused on other things. I will try and rebase. Feel free to close @apmoriarty if of no value....
I think LimitFields has actually gotten worse since this was written...but a few performance bugs were ironed out in the process. So maybe call it a draw.
No unit test to validate the fix?
Could you update the description with a brief overview of what use cases were missed?
index-only and term frequency fields handled in https://github.com/NationalSecurityAgency/datawave/pull/2968 composite fields handled in https://github.com/NationalSecurityAgency/datawave/pull/2982 memory considerations https://github.com/NationalSecurityAgency/datawave/pull/2987 outstanding: async aggregation
> It would be nice to add some additional commentary in the comments, but fundamentally this code looks ok. > > For future work, it would be nice to consider...