6e38bbb501e8eebfdd0ce097fc465c76ce2737fc
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 11
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Merge Attribute | 1 | ||
| Merge Attribute [keyBlock : Block, valueBlock : Block, hashTable : int[]] to mapBlock : AbstractMapBlock in class com.facebook.presto.spi.block.SingleMapBlock | From | To | |
| Change Parameter Type | 3 | ||
| Change Parameter Type hashTables : int[] to hashTables : HashTables in method public createMapBlockInternal(startOffset int, positionCount int, mapIsNull Optional<boolean[]>, offsets int[], keyBlock Block, valueBlock Block, hashTables HashTables, keyType Type, keyBlockNativeEquals MethodHandle, keyNativeHashCode MethodHandle, keyBlockHashCode MethodHandle) : MapBlock in class com.facebook.presto.spi.block.MapBlock | From | To | |
| Change Parameter Type hashTables : int[] to hashTables : HashTables in method private MapBlock(startOffset int, positionCount int, mapIsNull boolean[], offsets int[], keyBlock Block, valueBlock Block, hashTables HashTables, keyType Type, keyBlockNativeEquals MethodHandle, keyNativeHashCode MethodHandle, keyBlockHashCode MethodHandle) in class com.facebook.presto.spi.block.MapBlock | From | To | |
| Change Parameter Type hashTables : int[] to hashTables : HashTables in method public createMapBlockInternal(typeManager TypeManager, keyType Type, startOffset int, positionCount int, mapIsNull Optional<boolean[]>, offsets int[], keyBlock Block, valueBlock Block, hashTables HashTables) : Block in class com.facebook.presto.spi.type.MapType | From | To | |
| Rename Variable | 4 | ||
| Rename Variable newHashTable : int[] to newRawHashTables : int[] in method public copyRegion(position int, length int) : Block in class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Rename Variable hashTable : int[] to rawHashTables : int[] in method public copyPositions(positions int[], offset int, length int) : Block in class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Rename Variable newHashTable : int[] to newRawHashTables : int[] in method public getSingleValueBlock(position int) : Block in class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Rename Variable newHashTable : int[] to newRawHashTables : int[] in method public copyPositions(positions int[], offset int, length int) : Block in class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Pull Up Attribute | 1 | ||
| Pull Up Attribute private keyBlockHashCode : MethodHandle from class com.facebook.presto.spi.block.MapBlockBuilder to protected keyBlockHashCode : MethodHandle from class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Change Return Type | 3 | ||
| Change Return Type int[] to HashTables in method protected abstract getHashTables() : HashTables in class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Change Return Type int[] to HashTables in method protected getHashTables() : HashTables in class com.facebook.presto.spi.block.MapBlock | From | To | |
| Change Return Type int[] to HashTables in method protected getHashTables() : HashTables in class com.facebook.presto.spi.block.MapBlockBuilder | From | To | |
| Extract Variable | 8 | ||
| Extract Variable hashTableLength : int in method public writeBlock(blockEncodingSerde BlockEncodingSerde, sliceOutput SliceOutput, block Block) : void from class com.facebook.presto.spi.block.SingleMapBlockEncoding | From | To | |
| Extract Variable hashTableLength : int in method public writeBlock(blockEncodingSerde BlockEncodingSerde, sliceOutput SliceOutput, block Block) : void from class com.facebook.presto.spi.block.MapBlockEncoding | From | To | |
| Extract Variable hashTableLength : int in method public readBlock(blockEncodingSerde BlockEncodingSerde, sliceInput SliceInput) : Block from class com.facebook.presto.spi.block.SingleMapBlockEncoding | From | To | |
| Extract Variable expectedNewHashTableEntries : int in method public copyRegion(position int, length int) : Block from class com.facebook.presto.spi.block.AbstractMapBlock | From | To | |
| Extract Variable hashTableLength : int in method public readBlock(blockEncodingSerde BlockEncodingSerde, sliceInput SliceInput) : Block from class com.facebook.presto.spi.block.MapBlockEncoding | From | To | |
| Extract Variable previousAggregatedEntryCount : int in method private closeEntry(providedHashTable int[], providedHashTableOffset int) : BlockBuilder from class com.facebook.presto.spi.block.MapBlockBuilder | From | To | |
| Extract Variable aggregatedEntryCount : int in method private closeEntry(providedHashTable int[], providedHashTableOffset int) : BlockBuilder from class com.facebook.presto.spi.block.MapBlockBuilder | From | To | |
| Extract Variable hashTablesEntries : int in method public build() : Block from class com.facebook.presto.spi.block.MapBlockBuilder | From | To | |
| Change Attribute Type | 2 | ||
| Change Attribute Type hashTables : int[] to hashTables : HashTables in class com.facebook.presto.spi.block.MapBlock | From | To | |
| Change Attribute Type hashTables : int[] to hashTables : HashTables in class com.facebook.presto.spi.block.MapBlockBuilder | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter hashTables : int[] to rawHashTables : int[] in method private MapBlockBuilder(keyType Type, keyBlockNativeEquals MethodHandle, keyBlockEquals MethodHandle, keyNativeHashCode MethodHandle, keyBlockHashCode MethodHandle, blockBuilderStatus BlockBuilderStatus, keyBlockBuilder BlockBuilder, valueBlockBuilder BlockBuilder, offsets int[], mapIsNull boolean[], rawHashTables int[]) in class com.facebook.presto.spi.block.MapBlockBuilder | From | To |