fc0648d1c178ae4407e7a727d5a3bfd95af151ed
Files Diff
- Added: 2
 - Removed: 0
 - Renamed: 0
 - Modified: 10
 
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Push Down Method | 4 | ||
| Push Down Method public computeAssignments(splits Set<Split>) : Multimap<Node,Split> from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy to public computeAssignments(splits Set<Split>) : Multimap<Node,Split> from class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Push Down Method public SplitPlacementPolicy(nodeSelector NodeSelector, remoteTasks Supplier<List<RemoteTask>>) from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy to public DynamicSplitPlacementPolicy(nodeSelector NodeSelector, remoteTasks Supplier<? extends List<RemoteTask>>) from class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Push Down Method public allNodes() : List<Node> from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy to public allNodes() : List<Node> from class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Push Down Method public lockDownNodes() : void from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy to public lockDownNodes() : void from class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Extract Subclass | 1 | ||
| Extract Subclass com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy | From | To | |
| Change Parameter Type | 2 | ||
| Change Parameter Type nodePartitioningManager : NodePartitioningManager to partitioningCache : Function<PartitioningHandle,NodePartitionMap> in method private createStages(parent Optional<SqlStageExecution>, nextStageId AtomicInteger, locationFactory LocationFactory, plan StageExecutionPlan, nodeScheduler NodeScheduler, remoteTaskFactory RemoteTaskFactory, session Session, splitBatchSize int, partitioningCache Function<PartitioningHandle,NodePartitionMap>, executor ExecutorService, nodeTaskMap NodeTaskMap, stageSchedulers ImmutableMap.Builder<StageId,StageScheduler>, stageLinkages ImmutableMap.Builder<StageId,StageLinkage>) : List<SqlStageExecution> in class com.facebook.presto.execution.scheduler.SqlQueryScheduler | From | To | |
| Change Parameter Type remoteTasks : Supplier<List<RemoteTask>> to remoteTasks : Supplier<? extends List<RemoteTask>> in method public DynamicSplitPlacementPolicy(nodeSelector NodeSelector, remoteTasks Supplier<? extends List<RemoteTask>>) in class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Push Down Attribute | 1 | ||
| Push Down Attribute private nodeSelector : NodeSelector from class com.facebook.presto.execution.scheduler.SplitPlacementPolicy to private nodeSelector : NodeSelector from class com.facebook.presto.execution.scheduler.DynamicSplitPlacementPolicy | From | To | |
| Extract Variable | 1 | ||
| Extract Variable placementPolicy : SplitPlacementPolicy in method private createStages(parent Optional<SqlStageExecution>, nextStageId AtomicInteger, locationFactory LocationFactory, plan StageExecutionPlan, nodeScheduler NodeScheduler, remoteTaskFactory RemoteTaskFactory, session Session, splitBatchSize int, partitioningCache Function<PartitioningHandle,NodePartitionMap>, executor ExecutorService, nodeTaskMap NodeTaskMap, stageSchedulers ImmutableMap.Builder<StageId,StageScheduler>, stageLinkages ImmutableMap.Builder<StageId,StageLinkage>) : List<SqlStageExecution> from class com.facebook.presto.execution.scheduler.SqlQueryScheduler | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter nodePartitioningManager : NodePartitioningManager to partitioningCache : Function<PartitioningHandle,NodePartitionMap> in method private createStages(parent Optional<SqlStageExecution>, nextStageId AtomicInteger, locationFactory LocationFactory, plan StageExecutionPlan, nodeScheduler NodeScheduler, remoteTaskFactory RemoteTaskFactory, session Session, splitBatchSize int, partitioningCache Function<PartitioningHandle,NodePartitionMap>, executor ExecutorService, nodeTaskMap NodeTaskMap, stageSchedulers ImmutableMap.Builder<StageId,StageScheduler>, stageLinkages ImmutableMap.Builder<StageId,StageLinkage>) : List<SqlStageExecution> in class com.facebook.presto.execution.scheduler.SqlQueryScheduler | From | To |