5dbcd71cf2ce9261aa407e2f966c4ffd2ccbdebc
Files Diff
- Added: 4
- Removed: 12
- Renamed: 0
- Modified: 36
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 2 | ||
Rename Method public getHash() : Optional<Symbol> renamed to public getPartitionFunction() : Optional<PartitionFunctionBinding> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Rename Method public getPartitionBucket(partitionCount int, position int, page Page) : int renamed to public getPartition(partitionCount int, position int, page Page) : int in class com.facebook.presto.operator.HashGenerator | From | To | |
Change Parameter Type | 3 | ||
Change Parameter Type pagePartitionFunction : PagePartitionFunction to partition : int in method public withBuffer(bufferId TaskId, partition int) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Change Parameter Type buffers : Map<TaskId,PagePartitionFunction> to buffers : Map<TaskId,Integer> in method public withBuffers(buffers Map<TaskId,Integer>) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Change Parameter Type buffers : Map<TaskId,PagePartitionFunction> to buffers : Map<TaskId,Integer> in method public OutputBuffers(version long, noMoreBufferIds boolean, buffers Map<TaskId,Integer>) in class com.facebook.presto.OutputBuffers | From | To | |
Rename Variable | 2 | ||
Rename Variable pagePartitionFunction : PagePartitionFunction to partition : int in method public withBuffers(buffers Map<TaskId,Integer>) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Rename Variable bucket : int to partition : int in method public getPartition(partitionCount int, position int, page Page) : int in class com.facebook.presto.operator.HashGenerator | From | To | |
Change Return Type | 3 | ||
Change Return Type Optional<Symbol> to Optional<PartitionFunctionBinding> in method public getPartitionFunction() : Optional<PartitionFunctionBinding> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Return Type Map<TaskId,PagePartitionFunction> to Map<TaskId,Integer> in method public getBuffers() : Map<TaskId,Integer> in class com.facebook.presto.OutputBuffers | From | To | |
Change Return Type List<Node> to Map<Integer,Node> in method private generateRandomNodes(count int) : Map<Integer,Node> in class com.facebook.presto.execution.scheduler.TestFixedCountScheduler | From | To | |
Rename Class | 2 | ||
Rename Class com.facebook.presto.operator.PartitionedOutputOperator.PartitionFunction renamed to com.facebook.presto.operator.PartitionedOutputOperator.PagePartitioner | From | To | |
Rename Class com.facebook.presto.execution.scheduler.TestCurrentNodeScheduler renamed to com.facebook.presto.execution.scheduler.TestPartitionedOutputBufferManager | From | To | |
Move And Rename Class | 3 | ||
Move And Rename Class com.facebook.presto.PartitionedPagePartitionFunction moved and renamed to com.facebook.presto.sql.planner.PartitionFunctionHandle.RoundRobinPartitionFunction | From | To | |
Move And Rename Class com.facebook.presto.PagePartitionFunction moved and renamed to com.facebook.presto.operator.PartitionFunction | From | To | |
Move And Rename Class com.facebook.presto.UnpartitionedPagePartitionFunction moved and renamed to com.facebook.presto.sql.planner.PartitionFunctionHandle.SinglePartitionFunction | From | To | |
Rename Attribute | 1 | ||
Rename Attribute outputPartitioning : Optional<OutputPartitioning> to partitionFunction : Optional<PartitionFunctionHandle> in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
Change Variable Type | 5 | ||
Change Variable Type entry : Entry<TaskId,PagePartitionFunction> to entry : Entry<TaskId,Integer> in method public withBuffers(buffers Map<TaskId,Integer>) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Change Variable Type entry : Entry<TaskId,PagePartitionFunction> to entry : Entry<TaskId,Integer> in method public setOutputBuffers(newOutputBuffers OutputBuffers) : void in class com.facebook.presto.execution.SharedBuffer | From | To | |
Change Variable Type newBuffers : Map<TaskId,PagePartitionFunction> to newBuffers : Map<TaskId,Integer> in method public withBuffers(buffers Map<TaskId,Integer>) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Change Variable Type pagePartitionFunction : PagePartitionFunction to partition : int in method public withBuffers(buffers Map<TaskId,Integer>) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To | |
Change Variable Type nodes : ImmutableList.Builder<Node> to nodes : ImmutableMap.Builder<Integer,Node> in method private generateRandomNodes(count int) : Map<Integer,Node> in class com.facebook.presto.execution.scheduler.TestFixedCountScheduler | From | To | |
Change Attribute Type | 4 | ||
Change Attribute Type taskScheduler : Function<Node,RemoteTask> to taskScheduler : BiFunction<Node,Integer,RemoteTask> in class com.facebook.presto.execution.scheduler.FixedCountScheduler | From | To | |
Change Attribute Type buffers : Map<TaskId,PagePartitionFunction> to buffers : Map<TaskId,Integer> in class com.facebook.presto.OutputBuffers | From | To | |
Change Attribute Type partitionFunction : ListenableFuture<PartitionFunction> to partitionFunction : ListenableFuture<PagePartitioner> in class com.facebook.presto.operator.PartitionedOutputOperator | From | To | |
Change Attribute Type outputPartitioning : Optional<OutputPartitioning> to partitionFunction : Optional<PartitionFunctionHandle> in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
Rename Parameter | 1 | ||
Rename Parameter pagePartitionFunction : PagePartitionFunction to partition : int in method public withBuffer(bufferId TaskId, partition int) : OutputBuffers in class com.facebook.presto.OutputBuffers | From | To |