ac10c1b9c07b44e2c8379589dff5cb3adb74e4de
Files Diff
- Added: 20
- Removed: 5
- Renamed: 0
- Modified: 7
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 1 | ||
Rename Method private getHashPosition(rawHash long, mask int) : int renamed to private getBucketId(rawHash long, mask int) : int in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Move And Rename Class | 3 | ||
Move And Rename Class com.facebook.presto.operator.aggregation.state.HistogramStateFactory.GroupedState moved and renamed to com.facebook.presto.operator.aggregation.histogram.GroupedHistogramState | From | To | |
Move And Rename Class com.facebook.presto.operator.aggregation.state.HistogramStateFactory.SingleState moved and renamed to com.facebook.presto.operator.aggregation.histogram.SingleHistogramState | From | To | |
Move And Rename Class com.facebook.presto.operator.aggregation.TypedHistogram moved and renamed to com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Move Class | 3 | ||
Move Class com.facebook.presto.operator.aggregation.Histogram moved to com.facebook.presto.operator.aggregation.histogram.Histogram | From | To | |
Move Class com.facebook.presto.operator.aggregation.state.HistogramStateSerializer moved to com.facebook.presto.operator.aggregation.histogram.HistogramStateSerializer | From | To | |
Move Class com.facebook.presto.operator.aggregation.state.HistogramState moved to com.facebook.presto.operator.aggregation.histogram.HistogramState | From | To | |
Replace Variable With Attribute | 2 | ||
Replace Variable With Attribute otherCounts : LongBigArray to counts : LongBigArray in method public readAllValues(reader HistogramValueReader) : void in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Replace Variable With Attribute otherValues : Block to values : BlockBuilder in method public readAllValues(reader HistogramValueReader) : void in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Change Variable Type | 1 | ||
Change Variable Type otherValues : Block to values : BlockBuilder in method public readAllValues(reader HistogramValueReader) : void in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Extract Variable | 1 | ||
Extract Variable otherTypedHistogram : TypedHistogram in method public combine(state HistogramState, otherState HistogramState) : void from class com.facebook.presto.operator.aggregation.histogram.Histogram | From | To | |
Extract Interface | 1 | ||
Extract Interface com.facebook.presto.operator.aggregation.histogram.TypedHistogram from classes [com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram] | From | To | |
Change Attribute Type | 1 | ||
Change Attribute Type typedHistogram : TypedHistogram to typedHistogram : SingleTypedHistogram in class com.facebook.presto.operator.aggregation.histogram.SingleHistogramState | From | To | |
Extract Method | 3 | ||
Extract Method public createTestMetadataManager(catalogManager CatalogManager, featuresConfig FeaturesConfig) : MetadataManager extracted from public createTestMetadataManager(catalogManager CatalogManager) : MetadataManager in class com.facebook.presto.metadata.MetadataManager | From | To | |
Extract Method public readAllValues(reader HistogramValueReader) : void extracted from public addAll(other TypedHistogram) : void in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Extract Method private computeBucketCount(expectedSize int) : int extracted from public TypedHistogram(type Type, expectedSize int) in class com.facebook.presto.operator.aggregation.histogram.SingleTypedHistogram | From | To | |
Rename Parameter | 1 | ||
Rename Parameter value : TypedHistogram to typedHistogram : TypedHistogram in method public set(typedHistogram TypedHistogram) : void in class com.facebook.presto.operator.aggregation.histogram.HistogramState | From | To |