5dce2ff68cbd438f18bb6124f4a85461848e614d
Files Diff
- Added: 0
 - Removed: 0
 - Renamed: 0
 - Modified: 4
 
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Parameter Type | 6 | ||
| Change Parameter Type groups : List<List<Symbol>> to groups : List<List<String>> in method public groupingSet(groups List<List<String>>, groupIdAlias String, source PlanMatchPattern) : PlanMatchPattern in class com.facebook.presto.sql.planner.assertions.PlanMatchPattern | From | To | |
| Change Parameter Type identityMappings : Map<Symbol,Symbol> to identityMappings : Map<String,String> in method public GroupIdMatcher(groups List<List<String>>, identityMappings Map<String,String>, groupIdAlias String) in class com.facebook.presto.sql.planner.assertions.GroupIdMatcher | From | To | |
| Change Parameter Type groups : List<List<Symbol>> to groups : List<List<String>> in method public GroupIdMatcher(groups List<List<String>>, identityMappings Map<String,String>, groupIdAlias String) in class com.facebook.presto.sql.planner.assertions.GroupIdMatcher | From | To | |
| Change Parameter Type groupingSets : List<List<Symbol>> to groupingSets : List<List<String>> in method public aggregation(groupingSets List<List<String>>, aggregations Map<Optional<String>,ExpectedValueProvider<FunctionCall>>, masks Map<Symbol,Symbol>, groupId Optional<Symbol>, source PlanMatchPattern) : PlanMatchPattern in class com.facebook.presto.sql.planner.assertions.PlanMatchPattern | From | To | |
| Change Parameter Type actual : Collection<T> to actualSymbols : Collection<Symbol> in method package matches(expectedAliases Collection<String>, actualSymbols Collection<Symbol>, symbolAliases SymbolAliases) : boolean in class com.facebook.presto.sql.planner.assertions.AggregationMatcher | From | To | |
| Change Parameter Type groupingSets : List<List<Symbol>> to groupingSets : List<List<String>> in method public AggregationMatcher(groupingSets List<List<String>>, masks Map<Symbol,Symbol>, groupId Optional<Symbol>) in class com.facebook.presto.sql.planner.assertions.AggregationMatcher | From | To | |
| Change Variable Type | 4 | ||
| Change Variable Type groupByKeysSecond : List<Symbol> to groupByKeysSecond : List<String> in method public testMixedDistinctAggregationOptimizer() : void in class com.facebook.presto.sql.planner.optimizations.TestMixedDistinctAggregationOptimizer | From | To | |
| Change Variable Type symbol : T to symbol : Symbol in method package matches(expectedAliases Collection<String>, actualSymbols Collection<Symbol>, symbolAliases SymbolAliases) : boolean in class com.facebook.presto.sql.planner.assertions.AggregationMatcher | From | To | |
| Change Variable Type groups : ImmutableList.Builder<List<Symbol>> to groups : ImmutableList.Builder<List<String>> in method public testMixedDistinctAggregationOptimizer() : void in class com.facebook.presto.sql.planner.optimizations.TestMixedDistinctAggregationOptimizer | From | To | |
| Change Variable Type groupByKeysFirst : List<Symbol> to groupByKeysFirst : List<String> in method public testMixedDistinctAggregationOptimizer() : void in class com.facebook.presto.sql.planner.optimizations.TestMixedDistinctAggregationOptimizer | From | To | |
| Change Attribute Type | 3 | ||
| Change Attribute Type identityMappings : Map<Symbol,Symbol> to identityMappings : Map<String,String> in class com.facebook.presto.sql.planner.assertions.GroupIdMatcher | From | To | |
| Change Attribute Type groupingSets : List<List<Symbol>> to groupingSets : List<List<String>> in class com.facebook.presto.sql.planner.assertions.AggregationMatcher | From | To | |
| Change Attribute Type groups : List<List<Symbol>> to groups : List<List<String>> in class com.facebook.presto.sql.planner.assertions.GroupIdMatcher | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter actual : Collection<T> to actualSymbols : Collection<Symbol> in method package matches(expectedAliases Collection<String>, actualSymbols Collection<Symbol>, symbolAliases SymbolAliases) : boolean in class com.facebook.presto.sql.planner.assertions.AggregationMatcher | From | To |