36280057dc2cea86e3cb49636775c0ec11ee2205
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 14
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 3 | ||
| Rename Method public partitionedExchangeNullReplicate(id PlanNodeId, child PlanNode, partitionKey Symbol, hashSymbol Optional<Symbol>) : ExchangeNode renamed to public partitionedExchange(id PlanNodeId, child PlanNode, partitionFunction PartitionFunctionBinding) : ExchangeNode in class com.facebook.presto.sql.planner.plan.ExchangeNode | From | To | |
| Rename Method public getHashChannel() : Optional<Integer> renamed to public getHashColumn() : Optional<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Method public getPartitioningChannels() : List<Integer> renamed to public getPartitioningColumns() : List<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Parameter Type | 1 | ||
| Change Parameter Type partitioningChannels : List<Integer> to partitioningColumns : List<Symbol> in method public PartitionFunctionBinding(functionHandle PartitionFunctionHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>, replicateNulls boolean, partitionCount OptionalInt) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Variable | 1 | ||
| Rename Variable partitionKeys : Optional<List<Symbol>> to partitionFunction : Optional<PartitionFunctionBinding> in method public visitExchange(node ExchangeNode, context RewriteContext<Void>) : PlanNode in class com.facebook.presto.sql.planner.optimizations.UnaliasSymbolReferences.Rewriter | From | To | |
| Change Return Type | 2 | ||
| Change Return Type Optional<Integer> to Optional<Symbol> in method public getHashColumn() : Optional<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Return Type List<Integer> to List<Symbol> in method public getPartitioningColumns() : List<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Attribute | 3 | ||
| Rename Attribute hashSymbol : Optional<Symbol> to partitionFunction : Optional<PartitionFunctionBinding> in class com.facebook.presto.sql.planner.plan.ExchangeNode | From | To | |
| Rename Attribute hashChannel : Optional<Integer> to hashColumn : Optional<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Attribute partitioningChannels : List<Integer> to partitioningColumns : List<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Variable Type | 1 | ||
| Change Variable Type partitionKeys : Optional<List<Symbol>> to partitionFunction : Optional<PartitionFunctionBinding> in method public visitExchange(node ExchangeNode, context RewriteContext<Void>) : PlanNode in class com.facebook.presto.sql.planner.optimizations.UnaliasSymbolReferences.Rewriter | From | To | |
| Change Attribute Type | 4 | ||
| Change Attribute Type partitioningChannels : List<Integer> to partitioningColumns : List<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Change Attribute Type partitionFunction : Optional<PartitionFunctionHandle> to partitionFunction : Optional<PartitionFunctionBinding> in class com.facebook.presto.sql.planner.PlanFragmenter.FragmentProperties | From | To | |
| Change Attribute Type hashSymbol : Optional<Symbol> to partitionFunction : Optional<PartitionFunctionBinding> in class com.facebook.presto.sql.planner.plan.ExchangeNode | From | To | |
| Change Attribute Type hashChannel : Optional<Integer> to hashColumn : Optional<Symbol> in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter partitioningChannels : List<Integer> to partitioningColumns : List<Symbol> in method public PartitionFunctionBinding(functionHandle PartitionFunctionHandle, partitioningColumns List<Symbol>, hashColumn Optional<Symbol>, replicateNulls boolean, partitionCount OptionalInt) in class com.facebook.presto.sql.planner.PartitionFunctionBinding | From | To |