3bf6868e914097a97ce496b02c81259ae94f8854
Files Diff
- Added: 0
- Removed: 5
- Renamed: 0
- Modified: 13
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Merge Attribute | 1 | ||
| Merge Attribute [statsCalculator : TestingStatsCalculator, costCalculator : CostCalculator] to plan : PlanNode in class com.facebook.presto.sql.planner.iterative.rule.test.RuleAssert | From | To | |
| Change Parameter Type | 1 | ||
| Change Parameter Type statsProvider : StatsProvider to estimatedStatsAndCosts : StatsAndCosts in method private PlanPrinter(plan PlanNode, types TypeProvider, stageExecutionStrategy Optional<StageExecutionStrategy>, functionRegistry FunctionRegistry, estimatedStatsAndCosts StatsAndCosts, session Session, stats Optional<Map<PlanNodeId,PlanNodeStats>>, indent int, verbose boolean) in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter | From | To | |
| Change Return Type | 1 | ||
| Change Return Type String to Optional<String> in method private formatPlanNodeStatsAndCost(node PlanNode) : Optional<String> in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter.Visitor | From | To | |
| Merge Variable | 1 | ||
| Merge Variable [statsCalculator : StatsCalculator, sourceProvider : FragmentedPlanSourceProvider] to fragment : PlanFragment in method private formatFragment(functionRegistry FunctionRegistry, session Session, fragment PlanFragment, stageInfo Optional<StageInfo>, planNodeStats Optional<Map<PlanNodeId,PlanNodeStats>>, verbose boolean, allFragments List<PlanFragment>) : String in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter | From | To | |
| Merge Parameter | 1 | ||
| Merge Parameter [statsProvider : StatsProvider, costProvider : CostProvider] to estimatedStatsAndCosts : StatsAndCosts in method public textLogicalPlan(plan PlanNode, types TypeProvider, functionRegistry FunctionRegistry, estimatedStatsAndCosts StatsAndCosts, session Session, indent int) : String in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter | From | To | |
| Rename Attribute | 1 | ||
| Rename Attribute statsProvider : StatsProvider to estimatedStatsAndCosts : StatsAndCosts in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter.Visitor | From | To | |
| Change Attribute Type | 1 | ||
| Change Attribute Type statsProvider : StatsProvider to estimatedStatsAndCosts : StatsAndCosts in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter.Visitor | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter statsProvider : StatsProvider to estimatedStatsAndCosts : StatsAndCosts in method private PlanPrinter(plan PlanNode, types TypeProvider, stageExecutionStrategy Optional<StageExecutionStrategy>, functionRegistry FunctionRegistry, estimatedStatsAndCosts StatsAndCosts, session Session, stats Optional<Map<PlanNodeId,PlanNodeStats>>, indent int, verbose boolean) in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter | From | To |