d191d5944186bfab50ea1b91121fc7d37579344a
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 7
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 1 | ||
Rename Method public getSources() : List<PlanNode> renamed to public getPartitionedSourceNode() : PlanNode in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Rename Variable | 1 | ||
Rename Variable sourceIds : ImmutableSet.Builder<PlanNodeId> to remoteSourceNodes : ImmutableList.Builder<RemoteSourceNode> in method public PlanFragment(id PlanFragmentId, root PlanNode, symbols Map<Symbol,Type>, outputLayout List<Symbol>, distribution PlanDistribution, partitionedSource PlanNodeId, outputPartitioning OutputPartitioning, partitionBy List<Symbol>, hash Optional<Symbol>) in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Return Type | 1 | ||
Change Return Type List<PlanNode> to PlanNode in method public getPartitionedSourceNode() : PlanNode in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Replace Variable With Attribute | 1 | ||
Replace Variable With Attribute sourceIds : ImmutableSet.Builder<PlanNodeId> to remoteSourceNodes : List<RemoteSourceNode> in method private findRemoteSourceNodes(node PlanNode, builder Builder<RemoteSourceNode>) : void in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Rename Attribute | 1 | ||
Rename Attribute sourceIds : Set<PlanNodeId> to remoteSourceNodes : List<RemoteSourceNode> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Variable Type | 2 | ||
Change Variable Type sourceIds : ImmutableSet.Builder<PlanNodeId> to remoteSourceNodes : ImmutableList.Builder<RemoteSourceNode> in method public PlanFragment(id PlanFragmentId, root PlanNode, symbols Map<Symbol,Type>, outputLayout List<Symbol>, distribution PlanDistribution, partitionedSource PlanNodeId, outputPartitioning OutputPartitioning, partitionBy List<Symbol>, hash Optional<Symbol>) in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Variable Type sourceIds : ImmutableSet.Builder<PlanNodeId> to remoteSourceNodes : List<RemoteSourceNode> in method private findRemoteSourceNodes(node PlanNode, builder Builder<RemoteSourceNode>) : void in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Attribute Type | 1 | ||
Change Attribute Type sourceIds : Set<PlanNodeId> to remoteSourceNodes : List<RemoteSourceNode> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Extract Method | 1 | ||
Extract Method private findRemoteSourceNodes(node PlanNode, builder Builder<RemoteSourceNode>) : void extracted from public PlanFragment(id PlanFragmentId, root PlanNode, symbols Map<Symbol,Type>, outputLayout List<Symbol>, distribution PlanDistribution, partitionedSource PlanNodeId, outputPartitioning OutputPartitioning, partitionBy List<Symbol>, hash Optional<Symbol>) in class com.facebook.presto.sql.planner.PlanFragment | From | To |