Merge pull request #396 from apache/cleanupWarnings

Cleanup warnings

NO CODE CHANGES. Cleaned up Checkstyle warnings about extra empty lines.

THERE ARE NO CODE CHANGES! This is a cleanup of unnecessary Suppress
Warnings across the whole library.

The vast majority of these warnings were unnecessary
SuppressWarnings("javadoc") which were placed at the top of every test
file. We don't require javadocs of our test files and they should never
be published. This kind of suppression should be done at a global level
and
not in each file, which just added clutter to the code.

Release Process: Change pom version to 3.3.0-SNAPSHOT

Merge pull request #395 from apache/Final_cleanup_prior_to_release

Updated TestNG to 7.5. Removed unnecessary SuppressWarnings.  NO CODE

This is Jar Hell.

Turns out upgrading javac-errorprone to 2.11 requires JDK 11.  Had to
back off to 2.8.5.

Updated TestNG to 7.5. Removed unnecessary SuppressWarnings.  NO CODE
CHANGES

Merge pull request #394 from apache/OptimizeStep1

First pass speed optimization of the off-heap Kll sketch.

Remove unused method

First pass speed optimization of the off-heap Kll sketch.

Merge pull request #393 from apache/disallowUpdatableHeapify

Made the public heapify method throw if the given memory is in updatable

Make KllSketch public again as per request from Alex.

Made the public heapify method throw if the given memory is in updatable
format.

This commit removes the public toUpdatableByteArray() method from the
public classes KllDoubleSketch and KllFloatsSketch.

Merge pull request #392 from apache/RemoveExtraEmptyLines

This does not affect any code, This is a cosmetic fix to remove extra

This does not affect any code, This is a cosmetic fix to remove extra
blank lines.  I did find a couple of Javadoc spelling errors.

Merge pull request #391 from apache/DirectKllDoubleV2

Direct kll double v2

Added fast methods for creating serialized empty and singleItem sketch
byte arrays.

Merge branch 'master' into DirectKllDoubleV2

Merge branch 'master' of git@github.com:apache/datasketches-java.git

Fixing Security warnings (priority: HIGH) from CodeQL.

Create codeql-analysis.yml

Added parallel test for DoublesSketch

Optimizing Merge speed performance. Simplifying some rather obscure
code.

Some code optimization, removing unnecessary methods. Fixed some unit
tests that were not configured properly. Added unit tests.

Coce coverage is now about 96%.

Split up some tests into more focused files.

Added tests, cleaned up a few unused methods, improved some Javadocs.

Found a bug and made KllSketch public.

Made getMaxSerializedSizeBytes(...) static in both KllDoublesSketch and
KllFloatsSketch

Fix LGTM issue.

Interim 16

Interim 15

Interim 14

Fixed some tests. Other minor fixes.

Major refactoring completed.

Refactored KllDirectDoublesSketch and KllDirectFloatsSketch

Renamed KllFloatsSketch to KllHeapFloatsSketch

and KllDoublesSketch to KllHeapDoublesSketch

Added isSameResource(Memory).

Interim 13

Interim 12

This push fixes David's method naming issue.

This post fixes all the issues in the last set of reviews except two: 1)
I haven't renamed the memory dedicated "insert*" and "extract*" methods,
and 2) I haven't done the hierarchy refactoring along the lines we
discussed. I want check this in so that we can at least resolve all the
other conversations.

Removed mentions of m.

Moved public statics from KllPreambleUtil to KllSketch, added reset()
method.

Removed redundant "exclusive"

Fixed issues raised during review.

Fixing feedback from Review.

Found a few more miscellaneous items.

Fixed issues from latest comments.

Fix issues reported in PR Review, branch "Direct kll double" #360

Enabled the ability to grow into a larger configured Memory to reduce
the number of calls to the MemoryRequestServer.

Improved the KllHelper class to make it useful internally as well as to
interested users.

Fixed a small issue with getSerializedSizeBytes().

Remove multiple empty lines

Merge branch 'DirectKllDouble' of git@github.com:apache/datasketches-java.git into DirectKllDouble

Fixed the LGTM issues detected on the last push.

Fixed the LGTM issues detected on the last push.

Iteration 11, ready for review.

Interim 11.  Almost done.  KllDirectDoublesSketch working and tested.

Next: Create the KllDirectFloatsSketch and tests.

Interim 10

Interim 9

Interim 8

Interim 7 Fixes a bug introduced in Interim 6

Interim 6

Interim 5

interim 4

Interim 3

Interim 2

Interim 1

Merge pull request #389 from apache/RefactorKLL

Refactor kll

Minor changes in code documentation as a result of review.

More minor refactoring.

More refactoring and polishing:

Improved variable naming and method naming to be type specific where it
applied.

Rewrote toByteArray to use Memory.  This will make it easier to port to
Panama.

Moved class level javadocs to package-info because it is independent of
type.

Refactor KllDoublesSketch and KllFloatsSketch to extend a BaseKllSketch.

The base class contains all the common methods that don't depend on
float or double type. This reduced a lot of duplicate code.

Initial interim commit

Merge pull request #388 from apache/getUnionSize

Get union size

It turns out that the MurmurHash3v2Test.java file was the ONLY file in
the datasketches-java repo that had blank lines with extra spaces.

So I piggy-backed it here to avoid another PR.

Oops, forgot to comment out a print method.

Add two methods to Union:

getCurrentBytes() and getMaxUnionBytes()

Requested by Druid.

Update unit tests for KllDoubles and KllFloats

Merge pull request #387 from apache/KLLdoubles

KLL doubles

Fix typos: ≥ should have been ≥, etc.

Removed print statement in test. Fixed issue found by -P strict.

Create common KllHelper to contain common methods between float and
double helpers.

Updated SketchesCheckstyle.xml.

Improved Javadocs on InequalitySearch and GenericInequalitySearch.

This is the second phase of the creation of the KllDoublesSketch.

It runs all unit tests including the manual validation test.

This first commit prepares the current float-based classes.

No change in logic. Just some renaming of classes to be type specific.
Plus a few cosmetic code comment changes.

The KllFloatsValidationTest does work when properly enabled in that test
and in the KllFloatsHelper.

Merge pull request #385 from apache/FixAoDIntersect

Changed AoDIntersection check of matching numValues to an exception

Changed AoDIntersection check of matching numValues to an exception
instead of an assert.

Added Javadoc comments relevant to this requirement to Union, AnotB and
Intersection.

Merge pull request #384 from apache/FixKllComments

Fix kll comments

fix spelling, improved comments, etc.

No actual code changes.

Merge branch 'master' into CommonQuantilesCalculator

Merge pull request #383 from apache/3.1.X

3.1.x

Merge branch 'master' into 3.1.X

Update comments and code structure.

Minor changes to KLL.toString() format.

update toString to reveal more info.

Spelling and make easier to read.

No code changes.

Update pom with correct coordinates for git-commit-id-plugin.

Minor pom changes