This reflects how Druid actually was trying to use this method and ended up trying to simulate the second method incorrectly.
Removed the @Throws clause because it was duplicative of what is already in the Javadoc body. (Also, the CI/CD runner was throwing a class not found exception on SketchesArgumentException. I think it needed to be fully qualified. But I decided to remove these clauses anyway.)
QuantilesGenericAPI
extend new interface SketchPartitionLimits
SketchPartitionLimits
This new interface defines the methods required to implement the two new limit methods
getMaxPartitions()
getMinPartitionSizeItems()
Which can actually be implemented here as defaults.
/TEST/...
ItemsSketchTest
Propagation of change in ItemsSketchSortedView constructor change.
CrossCheckQuantilesTest
Propagation of change in ItemsSketchSortedView constructor change.
Class by Class description of the changes.
KllItemsSketch
Partitioner
ItemsSketch
GenericPartitionBoundaries
GenericSortedView
ItemsSketchSortedView
PartitionBoundaries
PartitioningFeature
SketchesArgumentException
. I think it needed to be fully qualified. But I decided to remove these clauses anyway.)QuantilesGenericAPI
SketchPartitionLimits
/TEST/...
ItemsSketchTest
CrossCheckQuantilesTest
PartitionBoundariesTest