e468a287edfcabd8baff1826a1bab953033e1bd4
Files Diff
- Added: 2
- Removed: 0
- Renamed: 0
- Modified: 21
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 1 | ||
| Rename Method public addInPredicates(node Node, inPredicates Set<InPredicate>) : void renamed to public recordSubqueries(node Node, expressionAnalysis ExpressionAnalysis) : void in class com.facebook.presto.sql.analyzer.Analysis | From | To | |
| Change Parameter Type | 1 | ||
| Change Parameter Type inPredicates : Set<InPredicate> to expressionAnalysis : ExpressionAnalysis in method public recordSubqueries(node Node, expressionAnalysis ExpressionAnalysis) : void in class com.facebook.presto.sql.analyzer.Analysis | From | To | |
| Rename Variable | 1 | ||
| Rename Variable outputTranslations : TranslationMap to translations : TranslationMap in method private copyTranslations(subPlan PlanBuilder) : TranslationMap in class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Extract Variable | 1 | ||
| Extract Variable type : JoinNode.Type in method public visitJoin(node JoinNode, context Context) : PlanWithProperties from class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Extract Method | 4 | ||
| Extract Method private copyTranslations(subPlan PlanBuilder) : TranslationMap extracted from private explicitCoercionSymbols(subPlan PlanBuilder, alreadyCoerced Iterable<Symbol>, uncoerced Iterable<? extends Expression>) : PlanBuilder in class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Extract Method private copyTranslations(subPlan PlanBuilder) : TranslationMap extracted from private appendSemiJoin(subPlan PlanBuilder, inPredicate InPredicate) : PlanBuilder in class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Extract Method private copyTranslations(subPlan PlanBuilder) : TranslationMap extracted from private appendProjections(subPlan PlanBuilder, expressions Iterable<Expression>) : PlanBuilder in class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Extract Method private copyTranslations(subPlan PlanBuilder) : TranslationMap extracted from private window(subPlan PlanBuilder, node QuerySpecification) : PlanBuilder in class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Inline Variable | 3 | ||
| Inline Variable inPredicates : Set<InPredicate> in method protected visitQuerySpecification(node QuerySpecification, context Void) : PlanBuilder from class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Inline Variable inPredicates : Set<InPredicate> in method protected visitQuery(query Query, context Void) : PlanBuilder from class com.facebook.presto.sql.planner.QueryPlanner | From | To | |
| Inline Variable subqueryInPredicates : Set<InPredicate> in method public analyzeExpression(session Session, metadata Metadata, accessControl AccessControl, sqlParser SqlParser, tupleDescriptor RelationType, analysis Analysis, approximateQueriesEnabled boolean, context AnalysisContext, expression Expression) : ExpressionAnalysis from class com.facebook.presto.sql.analyzer.ExpressionAnalyzer | From | To | |
| Rename Parameter | 2 | ||
| Rename Parameter inPredicates : Set<InPredicate> to expressionAnalysis : ExpressionAnalysis in method public recordSubqueries(node Node, expressionAnalysis ExpressionAnalysis) : void in class com.facebook.presto.sql.analyzer.Analysis | From | To | |
| Rename Parameter exceptionMessagePattern : String to regex : String in method private assertExceptionMessage(exception Exception, regex String) : void in class com.facebook.presto.tests.AbstractTestQueryFramework | From | To |