41cddded7fd916e63aeeb55484bd3b33c243b974
Files Diff
- Added: 2
- Removed: 0
- Renamed: 0
- Modified: 13
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Method | 3 | ||
Rename Method protected visitUnion(node Union, context AnalysisContext) : RelationType renamed to protected visitSetOperation(node SetOperation, context AnalysisContext) : RelationType in class com.facebook.presto.sql.analyzer.StatementAnalyzer | From | To | |
Rename Method private canonicalizeUnionSymbolMap(unionSymbolMap ListMultimap<Symbol,Symbol>) : ListMultimap<Symbol,Symbol> renamed to private canonicalizeSetOperationSymbolMap(setOperationSymbolMap ListMultimap<Symbol,Symbol>) : ListMultimap<Symbol,Symbol> in class com.facebook.presto.sql.planner.optimizations.UnaliasSymbolReferences.Rewriter | From | To | |
Rename Method protected visitUnion(node Union, context C) : R renamed to protected visitSetOperation(node SetOperation, context C) : R in class com.facebook.presto.sql.tree.DefaultTraversalVisitor | From | To | |
Change Parameter Type | 2 | ||
Change Parameter Type node : Union to node : SetOperation in method protected visitSetOperation(node SetOperation, context AnalysisContext) : RelationType in class com.facebook.presto.sql.analyzer.StatementAnalyzer | From | To | |
Change Parameter Type node : Union to node : SetOperation in method protected visitSetOperation(node SetOperation, context C) : R in class com.facebook.presto.sql.tree.DefaultTraversalVisitor | From | To | |
Rename Variable | 1 | ||
Rename Variable rewrittenUnion : UnionNode to rewrittenSetOperation : SetOperationNode in method private flattenSetOperation(node SetOperationNode, context RewriteContext<Boolean>, flattenedSources ImmutableList.Builder<PlanNode>, flattenedSymbolMap ImmutableListMultimap.Builder<Symbol,Symbol>) : void in class com.facebook.presto.sql.planner.optimizations.SetFlatteningOptimizer.Rewriter | From | To | |
Change Variable Type | 1 | ||
Change Variable Type rewrittenUnion : UnionNode to rewrittenSetOperation : SetOperationNode in method private flattenSetOperation(node SetOperationNode, context RewriteContext<Boolean>, flattenedSources ImmutableList.Builder<PlanNode>, flattenedSymbolMap ImmutableListMultimap.Builder<Symbol,Symbol>) : void in class com.facebook.presto.sql.planner.optimizations.SetFlatteningOptimizer.Rewriter | From | To | |
Extract Method | 3 | ||
Extract Method private visitSetOperation(node SetOperationNode, context Void) : Void extracted from public visitUnion(node UnionNode, context Void) : Void in class com.facebook.presto.sql.planner.PlanSanityChecker.Visitor | From | To | |
Extract Method private flattenSetOperation(node SetOperationNode, context RewriteContext<Boolean>, flattenedSources ImmutableList.Builder<PlanNode>, flattenedSymbolMap ImmutableListMultimap.Builder<Symbol,Symbol>) : void extracted from public visitUnion(node UnionNode, context RewriteContext<Boolean>) : PlanNode in class com.facebook.presto.sql.planner.optimizations.SetFlatteningOptimizer.Rewriter | From | To | |
Extract Method private rewriteSources(node SetOperationNode, context RewriteContext<Void>) : ImmutableList.Builder<PlanNode> extracted from public visitUnion(node UnionNode, context RewriteContext<Void>) : PlanNode in class com.facebook.presto.sql.planner.optimizations.UnaliasSymbolReferences.Rewriter | From | To | |
Extract And Move Method | 1 | ||
Extract And Move Method public visitIntersect(node IntersectNode, context C) : R extracted from public accept(visitor PlanVisitor<C,R>, context C) : R in class com.facebook.presto.sql.planner.plan.IntersectNode & moved to class com.facebook.presto.sql.planner.plan.PlanVisitor | From | To | |
Rename Parameter | 1 | ||
Rename Parameter unionSymbolMap : ListMultimap<Symbol,Symbol> to setOperationSymbolMap : ListMultimap<Symbol,Symbol> in method private canonicalizeSetOperationSymbolMap(setOperationSymbolMap ListMultimap<Symbol,Symbol>) : ListMultimap<Symbol,Symbol> in class com.facebook.presto.sql.planner.optimizations.UnaliasSymbolReferences.Rewriter | From | To |