af8c524d19117632082abe500ac6865a66457b46
Files Diff
- Added: 1
- Removed: 2
- Renamed: 0
- Modified: 6
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Move Method | 1 | ||
Move Method private calculateGrouping(groupId long, columns List<Integer>, groupingSetDescriptors List<List<Integer>>) : long from class com.facebook.presto.operator.scalar.GroupingOperationFunction to package calculateGrouping(groupId long, columns List<Integer>, groupingSetDescriptors List<List<Integer>>) : long from class com.facebook.presto.sql.planner.GroupingOperationRewriter | From | To | |
Rename Variable | 1 | ||
Rename Variable groupingSetOrdinals : List<List<Expression>> to groupingSetDescriptors : List<List<Integer>> in method public rewriteGroupingOperation(expression GroupingOperation, queryNode QuerySpecification, analysis Analysis, groupIdSymbol Optional<Symbol>) : Expression in class com.facebook.presto.sql.planner.GroupingOperationRewriter | From | To | |
Move Attribute | 2 | ||
Move Attribute public MAX_NUMBER_GROUPING_ARGUMENTS_BIGINT : int from class com.facebook.presto.operator.scalar.GroupingOperationFunction to private MAX_NUMBER_GROUPING_ARGUMENTS_BIGINT : int from class com.facebook.presto.sql.analyzer.ExpressionAnalyzer | From | To | |
Move Attribute public MAX_NUMBER_GROUPING_ARGUMENTS_INTEGER : int from class com.facebook.presto.operator.scalar.GroupingOperationFunction to private MAX_NUMBER_GROUPING_ARGUMENTS_INTEGER : int from class com.facebook.presto.sql.analyzer.ExpressionAnalyzer | From | To | |
Move And Rename Class | 1 | ||
Move And Rename Class com.facebook.presto.operator.scalar.TestGroupingOperationFunction moved and renamed to com.facebook.presto.sql.planner.TestGroupingOperationRewriter | From | To | |
Change Variable Type | 1 | ||
Change Variable Type groupingSetOrdinals : List<List<Expression>> to groupingSetDescriptors : List<List<Integer>> in method public rewriteGroupingOperation(expression GroupingOperation, queryNode QuerySpecification, analysis Analysis, groupIdSymbol Optional<Symbol>) : Expression in class com.facebook.presto.sql.planner.GroupingOperationRewriter | From | To |