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 |