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 |