b70ab6699009a5b2deb15ef184cb70c1693be9a5
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 18
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Merge Attribute | 1 | ||
| Merge Attribute [coordinatorOnly : boolean, partitioningProperties : Optional<Partitioning>] to streamPartitioning : Optional<Partitioning> in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Global | From | To | |
| Rename Method | 2 | ||
| Rename Method public partitioned(columns Set<Symbol>) : Partitioning renamed to public isPartitionedOn(partitioning PartitioningHandle, columns List<Symbol>) : boolean in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Rename Method public isDistributed() : boolean renamed to public isSingleNode() : boolean in class com.facebook.presto.sql.planner.optimizations.ActualProperties | From | To | |
| Move And Inline Method | 3 | ||
| Move And Inline Method public isHashPartitionedOn(columns List<Symbol>) : boolean moved from class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning to class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter & inlined to public visitJoin(node JoinNode, context Context) : PlanWithProperties | From | To | |
| Move And Inline Method public undistributed() : ActualProperties moved from class com.facebook.presto.sql.planner.optimizations.ActualProperties to class com.facebook.presto.sql.planner.optimizations.PropertyDerivations.Visitor & inlined to public visitIndexSource(node IndexSourceNode, context List<ActualProperties>) : ActualProperties | From | To | |
| Move And Inline Method public isHashPartitionedOn(columns List<Symbol>) : boolean moved from class com.facebook.presto.sql.planner.optimizations.ActualProperties to class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter & inlined to public visitJoin(node JoinNode, context Context) : PlanWithProperties | From | To | |
| Change Parameter Type | 1 | ||
| Change Parameter Type columns : Set<Symbol> to columns : List<Symbol> in method public isPartitionedOn(partitioning PartitioningHandle, columns List<Symbol>) : boolean in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Change Return Type | 1 | ||
| Change Return Type Partitioning to boolean in method public isPartitionedOn(partitioning PartitioningHandle, columns List<Symbol>) : boolean in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Rename Attribute | 1 | ||
| Rename Attribute hashingOrder : Optional<List<Symbol>> to partitioningHandle : PartitioningHandle in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Change Attribute Type | 2 | ||
| Change Attribute Type hashingOrder : Optional<List<Symbol>> to partitioningHandle : PartitioningHandle in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Change Attribute Type partitioningColumns : Set<Symbol> to partitioningColumns : List<Symbol> in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To | |
| Extract And Move Method | 1 | ||
| Extract And Move Method public getPartitioningHandle() : PartitioningHandle extracted from public isCoordinatorOnly() : boolean in class com.facebook.presto.sql.planner.optimizations.ActualProperties.Global & moved to class com.facebook.presto.sql.planner.optimizations.ActualProperties.Partitioning | From | To |