c8144b1ad1e78ee41972e51faca7e5daa06bb188
Files Diff
- Added: 1
- Removed: 0
- Renamed: 0
- Modified: 5
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Parameter Type | 4 | ||
| Change Parameter Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public BroadcastExchanger(buffers List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager) in class com.facebook.presto.operator.exchange.BroadcastExchanger | From | To | |
| Change Parameter Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public PartitioningExchanger(partitions List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager, types List<? extends Type>, partitionChannels List<Integer>, hashChannel Optional<Integer>) in class com.facebook.presto.operator.exchange.PartitioningExchanger | From | To | |
| Change Parameter Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public RandomExchanger(buffers List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager) in class com.facebook.presto.operator.exchange.RandomExchanger | From | To | |
| Change Parameter Type exchanger : Consumer<Page> to exchanger : LocalExchanger in method public LocalExchangeSink(types List<Type>, exchanger LocalExchanger, onFinish Consumer<LocalExchangeSink>) in class com.facebook.presto.operator.exchange.LocalExchangeSink | From | To | |
| Move Attribute | 3 | ||
| Move Attribute private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.LocalExchangeSink to private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.PartitioningExchanger | From | To | |
| Move Attribute private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.LocalExchangeSink to private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.RandomExchanger | From | To | |
| Move Attribute private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.LocalExchangeSink to private memoryManager : LocalExchangeMemoryManager from class com.facebook.presto.operator.exchange.BroadcastExchanger | From | To | |
| Rename Attribute | 3 | ||
| Rename Attribute memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.BroadcastExchanger | From | To | |
| Rename Attribute memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.PartitioningExchanger | From | To | |
| Rename Attribute memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.RandomExchanger | From | To | |
| Change Variable Type | 1 | ||
| Change Variable Type exchanger : Consumer<Page> to exchanger : LocalExchanger in method private createSink(factory LocalExchangeSinkFactory) : LocalExchangeSink in class com.facebook.presto.operator.exchange.LocalExchange | From | To | |
| Extract Interface | 1 | ||
| Extract Interface com.facebook.presto.operator.exchange.LocalExchanger from classes [com.facebook.presto.operator.exchange.BroadcastExchanger, com.facebook.presto.operator.exchange.PartitioningExchanger, com.facebook.presto.operator.exchange.RandomExchanger] | From | To | |
| Change Attribute Type | 5 | ||
| Change Attribute Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.BroadcastExchanger | From | To | |
| Change Attribute Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.PartitioningExchanger | From | To | |
| Change Attribute Type memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in class com.facebook.presto.operator.exchange.RandomExchanger | From | To | |
| Change Attribute Type exchangerSupplier : Supplier<Consumer<Page>> to exchangerSupplier : Supplier<LocalExchanger> in class com.facebook.presto.operator.exchange.LocalExchange | From | To | |
| Change Attribute Type exchanger : Consumer<Page> to exchanger : LocalExchanger in class com.facebook.presto.operator.exchange.LocalExchangeSink | From | To | |
| Rename Parameter | 3 | ||
| Rename Parameter memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public RandomExchanger(buffers List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager) in class com.facebook.presto.operator.exchange.RandomExchanger | From | To | |
| Rename Parameter memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public BroadcastExchanger(buffers List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager) in class com.facebook.presto.operator.exchange.BroadcastExchanger | From | To | |
| Rename Parameter memoryTracker : LongConsumer to memoryManager : LocalExchangeMemoryManager in method public PartitioningExchanger(partitions List<Consumer<PageReference>>, memoryManager LocalExchangeMemoryManager, types List<? extends Type>, partitionChannels List<Integer>, hashChannel Optional<Integer>) in class com.facebook.presto.operator.exchange.PartitioningExchanger | From | To |