91e5d906eaf130aa207ee673c57e76c82b3bec6d
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 13
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 2 | ||
Rename Method public extractOutputSymbols(planNode PlanNode) : Set<Symbol> renamed to public extractOutputVariables(planNode PlanNode) : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.SymbolsExtractor | From | To | |
Rename Method public extractOutputSymbols(planNode PlanNode, lookup Lookup) : Set<Symbol> renamed to public extractOutputVariables(planNode PlanNode, lookup Lookup) : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.SymbolsExtractor | From | To | |
Change Parameter Type | 1 | ||
Change Parameter Type symbols : Map<Symbol,Type> to variables : Set<VariableReferenceExpression> in method public PlanFragment(id PlanFragmentId, root PlanNode, variables Set<VariableReferenceExpression>, partitioning PartitioningHandle, tableScanSchedulingOrder List<PlanNodeId>, partitioningScheme PartitioningScheme, stageExecutionDescriptor StageExecutionDescriptor, outputTableWriterFragment boolean, statsAndCosts StatsAndCosts, jsonRepresentation Optional<String>) in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Rename Variable | 1 | ||
Rename Variable fragmentSymbolTypes : Map<Symbol,Type> to fragmentVariableTypes : Set<VariableReferenceExpression> in method private buildFragment(root PlanNode, properties FragmentProperties, fragmentId PlanFragmentId) : SubPlan in class com.facebook.presto.sql.planner.PlanFragmenter.Fragmenter | From | To | |
Change Return Type | 2 | ||
Change Return Type Set<Symbol> to Set<VariableReferenceExpression> in method public extractOutputVariables(planNode PlanNode, lookup Lookup) : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.SymbolsExtractor | From | To | |
Change Return Type Set<Symbol> to Set<VariableReferenceExpression> in method public extractOutputVariables(planNode PlanNode) : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.SymbolsExtractor | From | To | |
Rename Attribute | 1 | ||
Rename Attribute symbols : Map<Symbol,Type> to variables : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Change Variable Type | 1 | ||
Change Variable Type fragmentSymbolTypes : Map<Symbol,Type> to fragmentVariableTypes : Set<VariableReferenceExpression> in method private buildFragment(root PlanNode, properties FragmentProperties, fragmentId PlanFragmentId) : SubPlan in class com.facebook.presto.sql.planner.PlanFragmenter.Fragmenter | From | To | |
Extract Method | 2 | ||
Extract Method public fromVariables(variables Collection<VariableReferenceExpression>) : TypeProvider extracted from public fromVariables(variables VariableReferenceExpression...) : TypeProvider in class com.facebook.presto.sql.planner.TypeProvider | From | To | |
Extract Method private tableScan(id String, variables List<VariableReferenceExpression>) : TableScanNode extracted from private tableScan(id String, symbols String...) : TableScanNode in class com.facebook.presto.cost.TestCostCalculator | From | To | |
Change Attribute Type | 1 | ||
Change Attribute Type symbols : Map<Symbol,Type> to variables : Set<VariableReferenceExpression> in class com.facebook.presto.sql.planner.PlanFragment | From | To | |
Rename Parameter | 1 | ||
Rename Parameter symbols : Map<Symbol,Type> to variables : Set<VariableReferenceExpression> in method public PlanFragment(id PlanFragmentId, root PlanNode, variables Set<VariableReferenceExpression>, partitioning PartitioningHandle, tableScanSchedulingOrder List<PlanNodeId>, partitioningScheme PartitioningScheme, stageExecutionDescriptor StageExecutionDescriptor, outputTableWriterFragment boolean, statsAndCosts StatsAndCosts, jsonRepresentation Optional<String>) in class com.facebook.presto.sql.planner.PlanFragment | From | To |