fa3e1b761ab92b5e7fb0289222c37bee113fcf63
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 12
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Rename Variable | 1 | ||
Rename Variable taskSplits : Entry<Node,Collection<Split>> to node : Node in method private assignSplits(splitAssignment Multimap<Node,Split>, noMoreSplitsNotification Multimap<Node,Lifespan>) : Set<RemoteTask> in class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler | From | To | |
Inline Method | 2 | ||
Inline Method private handleNoMoreSplits() : ScheduleResult inlined to public schedule() : ScheduleResult in class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler | From | To | |
Move Attribute | 1 | ||
Move Attribute private pendingSplits : Set<Split> from class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler to public pendingSplits : Set<Split> from class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler.ScheduleGroup | From | To | |
Change Variable Type | 1 | ||
Change Variable Type taskSplits : Entry<Node,Collection<Split>> to node : Node in method private assignSplits(splitAssignment Multimap<Node,Split>, noMoreSplitsNotification Multimap<Node,Lifespan>) : Set<RemoteTask> in class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler | From | To | |
Extract Variable | 4 | ||
Extract Variable overallSplitAssignmentCount : int in method public schedule() : ScheduleResult from class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler | From | To | |
Extract Variable splits : ImmutableMultimap<PlanNodeId,Split> in method private assignSplits(splitAssignment Multimap<Node,Split>, noMoreSplitsNotification Multimap<Node,Lifespan>) : Set<RemoteTask> from class com.facebook.presto.execution.scheduler.SourcePartitionedScheduler | From | To | |
Extract Variable task : RemoteTask in method public scheduleSplits(node Node, splits Multimap<PlanNodeId,Split>, noMoreSplitsNotification Multimap<PlanNodeId,Lifespan>) : Set<RemoteTask> from class com.facebook.presto.execution.SqlStageExecution | From | To | |
Extract Variable splitSource : SplitSource in method private createStages(parent ExchangeLocationsConsumer, nextStageId AtomicInteger, locationFactory LocationFactory, plan StageExecutionPlan, nodeScheduler NodeScheduler, remoteTaskFactory RemoteTaskFactory, session Session, splitBatchSize int, partitioningCache Function<PartitioningHandle,NodePartitionMap>, nodePartitioningManager NodePartitioningManager, queryExecutor ExecutorService, schedulerExecutor ScheduledExecutorService, failureDetector FailureDetector, nodeTaskMap NodeTaskMap, stageSchedulers ImmutableMap.Builder<StageId,StageScheduler>, stageLinkages ImmutableMap.Builder<StageId,StageLinkage>) : List<SqlStageExecution> from class com.facebook.presto.execution.scheduler.SqlQueryScheduler | From | To | |
Change Attribute Type | 1 | ||
Change Attribute Type sourcePartitionedSchedulers : Queue<SourcePartitionedScheduler> to sourcePartitionedSchedulers : List<SourcePartitionedScheduler> in class com.facebook.presto.execution.scheduler.FixedSourcePartitionedScheduler | From | To |