e4f7d5c616ae69ff85f160d523c3091b56b10f76
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 11
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Parameter Type | 2 | ||
| Change Parameter Type valueIsNull : SliceOutput to valueIsNull : boolean[] in method private ArrayBlockBuilder(blockBuilderStatus BlockBuilderStatus, values BlockBuilder, offsets int[], valueIsNull boolean[]) in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Change Parameter Type offsets : SliceOutput to offsets : int[] in method private ArrayBlockBuilder(blockBuilderStatus BlockBuilderStatus, values BlockBuilder, offsets int[], valueIsNull boolean[]) in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Rename Variable | 4 | ||
| Rename Variable j : int to elementIndex : int in method public copyPositions(positions List<Integer>) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Rename Variable countNewOffset : int to newPosition : int in method public copyPositions(positions List<Integer>) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Rename Variable positionEndOffset : int to valuesEndOffset : int in method public copyPositions(positions List<Integer>) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Rename Variable positionStartOffset : int to valuesStartOffset : int in method public copyPositions(positions List<Integer>) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Inline Method | 2 | ||
| Inline Method private getRegion(position int, length int, compact boolean) : Block inlined to public copyRegion(position int, length int) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Inline Method private getRegion(position int, length int, compact boolean) : Block inlined to public getRegion(position int, length int) : Block in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Change Return Type | 6 | ||
| Change Return Type Slice to int[] in method protected getOffsets() : int[] in class com.facebook.presto.spi.block.ArrayBlock | From | To | |
| Change Return Type Slice to boolean[] in method protected abstract getValueIsNull() : boolean[] in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Change Return Type Slice to int[] in method protected getOffsets() : int[] in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Change Return Type Slice to int[] in method protected abstract getOffsets() : int[] in class com.facebook.presto.spi.block.AbstractArrayBlock | From | To | |
| Change Return Type Slice to boolean[] in method protected getValueIsNull() : boolean[] in class com.facebook.presto.spi.block.ArrayBlock | From | To | |
| Change Return Type Slice to boolean[] in method protected getValueIsNull() : boolean[] in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Extract Method | 1 | ||
| Extract Method private updateDataSize() : void extracted from private ArrayBlockBuilder(blockBuilderStatus BlockBuilderStatus, values BlockBuilder, offsets SliceOutput, valueIsNull SliceOutput) in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Change Attribute Type | 4 | ||
| Change Attribute Type offsets : Slice to offsets : int[] in class com.facebook.presto.spi.block.ArrayBlock | From | To | |
| Change Attribute Type valueIsNull : SliceOutput to valueIsNull : boolean[] in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To | |
| Change Attribute Type valueIsNull : Slice to valueIsNull : boolean[] in class com.facebook.presto.spi.block.ArrayBlock | From | To | |
| Change Attribute Type offsets : SliceOutput to offsets : int[] in class com.facebook.presto.spi.block.ArrayBlockBuilder | From | To |