849016ac56cc29955870d3cdf0ae4dc354de01b7
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 6
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 4 | ||
| Rename Method public hashPartitioned(columns List<Symbol>) : PreferredProperties renamed to public partitionedWithNullsReplicated(partitioning Partitioning) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Rename Method public getHashingOrder() : Optional<List<Symbol>> renamed to public getPartitioning() : Optional<Partitioning> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Rename Method public hashPartitionedWithNullsReplicated(columns List<Symbol>) : PreferredProperties renamed to public partitionedWithNullsReplicated(columns Set<Symbol>) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Rename Method public hashPartitioned(columns List<Symbol>) : PartitioningProperties renamed to public partitioned(partitioning Partitioning) : PartitioningProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Change Parameter Type | 3 | ||
| Change Parameter Type columns : List<Symbol> to partitioning : Partitioning in method public partitionedWithNullsReplicated(partitioning Partitioning) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Change Parameter Type columns : List<Symbol> to columns : Set<Symbol> in method public partitionedWithNullsReplicated(columns Set<Symbol>) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Change Parameter Type columns : List<Symbol> to partitioning : Partitioning in method public partitioned(partitioning Partitioning) : PartitioningProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Rename Variable | 3 | ||
| Rename Variable newHashingOrder : Optional<List<Symbol>> to newPartitioning : Optional<Partitioning> in method public translate(translator Function<Symbol,Optional<Symbol>>) : Optional<PartitioningProperties> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Rename Variable sourceToFilteringSource : SetMultimap<Symbol,Symbol> to builder : ImmutableSetMultimap.Builder<T,T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Rename Variable partitioningPreference : PreferredProperties.PartitioningProperties to parentPartitioningPreference : PreferredProperties.PartitioningProperties in method public visitUnion(node UnionNode, context Context) : PlanWithProperties in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Return Type | 1 | ||
| Change Return Type Optional<List<Symbol>> to Optional<Partitioning> in method public getPartitioning() : Optional<Partitioning> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Parameterize Variable | 2 | ||
| Parameterize Variable sourceSymbols : List<Symbol> to keys : List<T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Parameterize Variable filteringSourceSymbols : List<Symbol> to values : List<T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Rename Attribute | 1 | ||
| Rename Attribute hashingOrder : Optional<List<Symbol>> to partitioning : Optional<Partitioning> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Change Variable Type | 4 | ||
| Change Variable Type sourceSymbols : List<Symbol> to keys : List<T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Variable Type newHashingOrder : Optional<List<Symbol>> to newPartitioning : Optional<Partitioning> in method public translate(translator Function<Symbol,Optional<Symbol>>) : Optional<PartitioningProperties> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Change Variable Type sourceToFilteringSource : SetMultimap<Symbol,Symbol> to builder : ImmutableSetMultimap.Builder<T,T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Variable Type filteringSourceSymbols : List<Symbol> to values : List<T> in method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Change Attribute Type | 1 | ||
| Change Attribute Type hashingOrder : Optional<List<Symbol>> to partitioning : Optional<Partitioning> in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Extract Method | 3 | ||
| Extract Method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> extracted from public visitJoin(node JoinNode, context Context) : PlanWithProperties in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Extract Method private createMapping(keys List<T>, values List<T>) : SetMultimap<T,T> extracted from public visitSemiJoin(node SemiJoinNode, context Context) : PlanWithProperties in class com.facebook.presto.sql.planner.optimizations.AddExchanges.Rewriter | From | To | |
| Extract Method public partitioned(partitioning Partitioning) : PreferredProperties extracted from public hashPartitioned(columns List<Symbol>) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To | |
| Rename Parameter | 2 | ||
| Rename Parameter columns : List<Symbol> to partitioning : Partitioning in method public partitioned(partitioning Partitioning) : PartitioningProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties.PartitioningProperties | From | To | |
| Rename Parameter columns : List<Symbol> to partitioning : Partitioning in method public partitionedWithNullsReplicated(partitioning Partitioning) : PreferredProperties in class com.facebook.presto.sql.planner.optimizations.PreferredProperties | From | To |