24771e5b9b1765f498d717a8a600ca7d2ee80bfc
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 4
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Change Parameter Type | 2 | ||
Change Parameter Type outputs : String to outputs : List<Symbol> in method public NodeRepresentation(id PlanNodeId, type String, identifier String, outputs List<Symbol>, stats Optional<PlanNodeStats>, estimatedStats List<PlanNodeStatsEstimate>, estimatedCost List<PlanNodeCostEstimate>, children List<PlanNodeId>) in class com.facebook.presto.sql.planner.planPrinter.NodeRepresentation | From | To | |
Change Parameter Type output : NodeRepresentation to output : StringBuilder in method private printDistributions(output StringBuilder, stats PlanNodeStats) : void in class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method | 6 | ||
Move Method private formatDouble(value double) : String from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to package formatDouble(value double) : String from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method private formatEstimateAsDataSize(value double) : String from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to private formatEstimateAsDataSize(value double) : String from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method private formatPositions(positions long) : String from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to package formatPositions(positions long) : String from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method private printDistributions(output NodeRepresentation, stats PlanNodeStats) : void from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to private printDistributions(output StringBuilder, stats PlanNodeStats) : void from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method private formatAsLong(value double) : String from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to private formatAsLong(value double) : String from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Move Method private translateOperatorTypes(operators Set<String>) : Map<String,String> from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to private translateOperatorTypes(operators Set<String>) : Map<String,String> from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Rename Variable | 1 | ||
Rename Variable statsAndCost : String to stats : String in method private writeTextOutput(output StringBuilder, plan PlanRepresentation, level int, node NodeRepresentation) : String in class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Change Return Type | 2 | ||
Change Return Type String to List<Symbol> in method public getOutputs() : List<Symbol> in class com.facebook.presto.sql.planner.planPrinter.NodeRepresentation | From | To | |
Change Return Type String to Optional<PlanNodeStats> in method public getStats() : Optional<PlanNodeStats> in class com.facebook.presto.sql.planner.planPrinter.NodeRepresentation | From | To | |
Move Attribute | 1 | ||
Move Attribute private verbose : boolean from class com.facebook.presto.sql.planner.planPrinter.PlanPrinter to private verbose : boolean from class com.facebook.presto.sql.planner.planPrinter.TextRenderer | From | To | |
Extract Method | 1 | ||
Extract Method public addNode(rootNode PlanNode, name String, identifier String, allNodes List<PlanNodeId>, children List<PlanNode>) : NodeRepresentation extracted from public addNode(node PlanNode, name String, identifier String, children List<PlanNode>) : NodeRepresentation in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter.Visitor | From | To | |
Change Attribute Type | 2 | ||
Change Attribute Type outputs : String to outputs : List<Symbol> in class com.facebook.presto.sql.planner.planPrinter.NodeRepresentation | From | To | |
Change Attribute Type stats : StringBuilder to stats : Optional<PlanNodeStats> in class com.facebook.presto.sql.planner.planPrinter.NodeRepresentation | From | To | |
Rename Parameter | 1 | ||
Rename Parameter node : PlanNode to rootNode : PlanNode in method public addNode(rootNode PlanNode, name String, identifier String, allNodes List<PlanNodeId>, children List<PlanNode>) : NodeRepresentation in class com.facebook.presto.sql.planner.planPrinter.PlanPrinter.Visitor | From | To |