270fb6513729458e6650a26b89ac83cd0bad61ee
Files Diff
- Added: 1
- Removed: 0
- Renamed: 0
- Modified: 6
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Move And Rename Method | 1 | ||
| Move And Rename Method private calculateJoinCost(probe PlanNode, build PlanNode, replicated boolean) : PlanNodeCostEstimate from class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges.ExchangeCostEstimator to public calculateJoinExchangeCost(probe PlanNode, build PlanNode, stats StatsProvider, types TypeProvider, replicated boolean, estimatedSourceDistributedTaskCount int) : PlanNodeCostEstimate from class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Change Parameter Type | 2 | ||
| Change Parameter Type numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in method public CostCalculatorUsingExchanges(taskCountEstimator TaskCountEstimator) in class com.facebook.presto.cost.CostCalculatorUsingExchanges | From | To | |
| Change Parameter Type numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in method public CostCalculatorWithEstimatedExchanges(costCalculator CostCalculator, taskCountEstimator TaskCountEstimator) in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Rename Variable | 1 | ||
| Rename Variable sourceSymbols : List<Symbol> to source : PlanNode in method public visitAggregation(node AggregationNode, context Void) : PlanNodeCostEstimate in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges.ExchangeCostEstimator | From | To | |
| Move Attribute | 2 | ||
| Move Attribute private numberOfNodes : IntSupplier from class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges to private numberOfNodes : IntSupplier from class com.facebook.presto.cost.TaskCountEstimator | From | To | |
| Move Attribute private numberOfNodes : IntSupplier from class com.facebook.presto.cost.CostCalculatorUsingExchanges to private numberOfNodes : IntSupplier from class com.facebook.presto.cost.TaskCountEstimator | From | To | |
| Extract Class | 4 | ||
| Extract Class com.facebook.presto.cost.TaskCountEstimator from class com.facebook.presto.cost.CostCalculatorUsingExchanges.CostEstimator | From | To | |
| Extract Class com.facebook.presto.cost.TaskCountEstimator from class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Extract Class com.facebook.presto.cost.TaskCountEstimator from class com.facebook.presto.cost.CostCalculatorUsingExchanges | From | To | |
| Extract Class com.facebook.presto.cost.TaskCountEstimator from class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges.ExchangeCostEstimator | From | To | |
| Rename Attribute | 3 | ||
| Rename Attribute numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Rename Attribute numberOfNodes : int to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorUsingExchanges.CostEstimator | From | To | |
| Rename Attribute numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorUsingExchanges | From | To | |
| Change Variable Type | 1 | ||
| Change Variable Type sourceSymbols : List<Symbol> to source : PlanNode in method public visitAggregation(node AggregationNode, context Void) : PlanNodeCostEstimate in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges.ExchangeCostEstimator | From | To | |
| Extract Method | 1 | ||
| Extract Method private calculateJoinOutputCost(join PlanNode) : PlanNodeCostEstimate extracted from private calculateJoinCost(join PlanNode, probe PlanNode, build PlanNode, replicated boolean) : PlanNodeCostEstimate in class com.facebook.presto.cost.CostCalculatorUsingExchanges.CostEstimator | From | To | |
| Change Attribute Type | 3 | ||
| Change Attribute Type numberOfNodes : int to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorUsingExchanges.CostEstimator | From | To | |
| Change Attribute Type numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorUsingExchanges | From | To | |
| Change Attribute Type numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Extract And Move Method | 1 | ||
| Extract And Move Method public calculateJoinInputCost(probe PlanNode, build PlanNode, stats StatsProvider, types TypeProvider, replicated boolean, estimatedSourceDistributedTaskCount int) : PlanNodeCostEstimate extracted from private calculateJoinCost(join PlanNode, probe PlanNode, build PlanNode, replicated boolean) : PlanNodeCostEstimate in class com.facebook.presto.cost.CostCalculatorUsingExchanges.CostEstimator & moved to class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Rename Parameter | 2 | ||
| Rename Parameter numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in method public CostCalculatorWithEstimatedExchanges(costCalculator CostCalculator, taskCountEstimator TaskCountEstimator) in class com.facebook.presto.cost.CostCalculatorWithEstimatedExchanges | From | To | |
| Rename Parameter numberOfNodes : IntSupplier to taskCountEstimator : TaskCountEstimator in method public CostCalculatorUsingExchanges(taskCountEstimator TaskCountEstimator) in class com.facebook.presto.cost.CostCalculatorUsingExchanges | From | To |