be6376f8df4c266e452220bacf55c302d340d1ae
Files Diff
- Added: 0
- Removed: 1
- Renamed: 0
- Modified: 21
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 2 | ||
| Rename Method public getDistribution() : PlanDistribution renamed to public getPartitioningHandle() : PartitioningHandle in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
| Rename Method public getFunctionHandle() : PartitionFunctionHandle renamed to public getPartitioningHandle() : PartitioningHandle in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Parameter Type | 5 | ||
| Change Parameter Type functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>, replicateNulls boolean, bucketToPartition Optional<int[]>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Parameter Type partitioningHandle : Optional<PartitioningHandle> to partitioningHandle : PartitioningHandle in method public StageExecutionPlan(fragment PlanFragment, dataSource Optional<SplitSource>, partitioningHandle PartitioningHandle, subStages List<StageExecutionPlan>) in class com.facebook.presto.sql.planner.StageExecutionPlan | From | To | |
| Change Parameter Type distribution : PlanDistribution to partitioning : PartitioningHandle in method public PlanFragment(id PlanFragmentId, root PlanNode, symbols Map<Symbol,Type>, outputLayout List<Symbol>, partitioning PartitioningHandle, partitionedSource PlanNodeId, partitionFunction Optional<PartitionFunctionBinding>) in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
| Change Parameter Type functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Parameter Type functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Return Type | 3 | ||
| Change Return Type PartitionFunctionHandle to PartitioningHandle in method public getPartitioningHandle() : PartitioningHandle in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Return Type PlanDistribution to PartitioningHandle in method public getPartitioningHandle() : PartitioningHandle in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
| Change Return Type Optional<PartitioningHandle> to PartitioningHandle in method public getPartitioningHandle() : PartitioningHandle in class com.facebook.presto.sql.planner.StageExecutionPlan | From | To | |
| Move Class | 3 | ||
| Move Class com.facebook.presto.sql.planner.PartitionFunctionHandle.HashBucketFunction moved to com.facebook.presto.sql.planner.SystemPartitioningHandle.SystemPartitionFunction.HashBucketFunction | From | To | |
| Move Class com.facebook.presto.sql.planner.PartitionFunctionHandle.RoundRobinBucketFunction moved to com.facebook.presto.sql.planner.SystemPartitioningHandle.SystemPartitionFunction.RoundRobinBucketFunction | From | To | |
| Move Class com.facebook.presto.sql.planner.PartitionFunctionHandle.SingleBucketFunction moved to com.facebook.presto.sql.planner.SystemPartitioningHandle.SystemPartitionFunction.SingleBucketFunction | From | To | |
| Rename Attribute | 4 | ||
| Rename Attribute distribution : Optional<PlanDistribution> to partitioningHandle : Optional<PartitioningHandle> in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
| Rename Attribute planDistribution : PlanDistribution to partitioning : SystemPartitioning in class com.facebook.presto.sql.planner.SystemPartitioningHandle | From | To | |
| Rename Attribute distribution : PlanDistribution to partitioning : PartitioningHandle in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
| Rename Attribute functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Extract Variable | 3 | ||
| Extract Variable partitionFunction : PartitionFunctionBinding in method public visitExchange(exchange ExchangeNode, context RewriteContext<FragmentProperties>) : PlanNode from class com.facebook.presto.sql.planner.PlanFragmenter.Fragmenter | From | To | |
| Extract Variable partitionFunction : PartitionFunctionBinding in method public visitAggregation(node AggregationNode, context Context) : PlanWithProperties from class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Extract Variable partitionFunction : PartitionFunctionBinding in method public visitWindow(node WindowNode, context Context) : PlanWithProperties from class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Attribute Type | 5 | ||
| Change Attribute Type partitioningHandle : Optional<PartitioningHandle> to partitioningHandle : PartitioningHandle in class com.facebook.presto.sql.planner.StageExecutionPlan | From | To | |
| Change Attribute Type distribution : Optional<PlanDistribution> to partitioningHandle : Optional<PartitioningHandle> in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
| Change Attribute Type planDistribution : PlanDistribution to partitioning : SystemPartitioning in class com.facebook.presto.sql.planner.SystemPartitioningHandle | From | To | |
| Change Attribute Type distribution : PlanDistribution to partitioning : PartitioningHandle in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
| Change Attribute Type functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Extract And Move Method | 2 | ||
| Extract And Move Method public getPartitionFunction(partitionChannelTypes List<Type>, isHashPrecomputed boolean, bucketToPartition int[]) : PartitionFunction extracted from public getPartitionFunction(session Session, functionBinding PartitionFunctionBinding, partitionChannelTypes List<Type>) : PartitionFunction in class com.facebook.presto.sql.planner.NodePartitioningManager & moved to class com.facebook.presto.sql.planner.SystemPartitioningHandle | From | To | |
| Extract And Move Method public getNodePartitionMap(session Session, nodeScheduler NodeScheduler) : NodePartitionMap extracted from public getNodePartitioningMap(session Session, partitioningHandle PartitioningHandle) : NodePartitionMap in class com.facebook.presto.sql.planner.NodePartitioningManager & moved to class com.facebook.presto.sql.planner.SystemPartitioningHandle | From | To | |
| Rename Parameter | 4 | ||
| Rename Parameter functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>, replicateNulls boolean, bucketToPartition Optional<int[]>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Parameter distribution : PlanDistribution to partitioning : PartitioningHandle in method public PlanFragment(id PlanFragmentId, root PlanNode, symbols Map<Symbol,Type>, outputLayout List<Symbol>, partitioning PartitioningHandle, partitionedSource PlanNodeId, partitionFunction Optional<PartitionFunctionBinding>) in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
| Rename Parameter functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Parameter functionHandle : PartitionFunctionHandle to partitioningHandle : PartitioningHandle in method public PartitionFunctionBinding(partitioningHandle PartitioningHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To |