87ed3f0782bb69527d5930e16596fd9f199a2f91
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 6
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 1 | ||
| Rename Method public getPartitioningColumns() : Optional<Set<Symbol>> renamed to public getPartitioningProperties() : Optional<PartitioningPreferences> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Inline Method | 2 | ||
| Inline Method private planAggregation(node AggregationNode, preferredChildProperties PreferredProperties, addExchange Function<PlanNode,PlanNode>) : PlanWithProperties inlined to public visitAggregation(node AggregationNode, preferred PreferredProperties) : PlanWithProperties in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Return Type | 1 | ||
| Change Return Type Optional<Set<Symbol>> to Optional<PartitioningPreferences> in method public getPartitioningProperties() : Optional<PartitioningPreferences> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Move Attribute | 1 | ||
| Move Attribute private partitioningColumns : Optional<Set<Symbol>> from class com.facebook.presto.sql.planner.optimizations.PreferredProperties to private partitioningColumns : Optional<Set<Symbol>> from class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningPreferences | From | To | |
| Rename Attribute | 1 | ||
| Rename Attribute hasPartitioningRequirements : boolean to partitioningRequirements : Optional<PartitioningPreferences> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Change Variable Type | 1 | ||
| Change Variable Type constants : Map<Symbol,Object> to constants : Map<ColumnHandle,Object> in method public visitTableScan(node TableScanNode, inputProperties List<ActualProperties>) : ActualProperties in class com.facebook.presto.sql.planner.optimizations.PropertyDerivations.Visitor | From | To | |
| Extract Variable | 2 | ||
| Extract Variable constantAppendedLocalProperties : List<LocalProperty<ColumnHandle>> in method public visitTableScan(node TableScanNode, inputProperties List<ActualProperties>) : ActualProperties from class com.facebook.presto.sql.planner.optimizations.PropertyDerivations.Visitor | From | To | |
| Extract Variable updatedConstants : Map<Symbol,Object> in method private ActualProperties(partitioningColumns Optional<Set<Symbol>>, hashingColumns Optional<List<Symbol>>, localProperties List<? extends LocalProperty<Symbol>>, partitioned boolean, coordinatorOnly boolean, constants Map<Symbol,Object>) from class com.facebook.presto.sql.planner.optimizations.ActualProperties | From | To | |
| Change Attribute Type | 1 | ||
| Change Attribute Type hasPartitioningRequirements : boolean to partitioningRequirements : Optional<PartitioningPreferences> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Extract And Move Method | 2 | ||
| Extract And Move Method public translate(properties List<? extends LocalProperty<X>>, translator Function<X,Optional<Y>>) : List<LocalProperty<Y>> extracted from public visitTableScan(node TableScanNode, inputProperties List<ActualProperties>) : ActualProperties in class com.facebook.presto.sql.planner.optimizations.PropertyDerivations.Visitor & moved to class com.facebook.presto.sql.planner.optimizations.LocalProperties | From | To | |
| Extract And Move Method public translate(properties List<? extends LocalProperty<X>>, translator Function<X,Optional<Y>>) : List<LocalProperty<Y>> extracted from public visitProject(node ProjectNode, inputProperties List<ActualProperties>) : ActualProperties in class com.facebook.presto.sql.planner.optimizations.PropertyDerivations.Visitor & moved to class com.facebook.presto.sql.planner.optimizations.LocalProperties | From | To |