11899c4c617fed41ee6ee9c4f33f06f0db245982
Files Diff
- Added: 0
- Removed: 6
- Renamed: 0
- Modified: 6
Refactorings reported:
Refactoring Name | Occurences | ||
---|---|---|---|
Move And Inline Method | 1 | ||
Move And Inline Method public getLabelsList() : List<SingularityMesosTaskLabel> moved from class com.hubspot.singularity.SingularityDeploy to class com.hubspot.singularity.mesos.SingularityMesosTaskBuilder & inlined to public buildTask(offer Protos.Offer, availableResources List<Resource>, taskRequest SingularityTaskRequest, desiredTaskResources Resources, desiredExecutorResources Resources) : SingularityTask | From | To | |
Change Parameter Type | 3 | ||
Change Parameter Type labels : Optional<SingularityMesosTaskLabels> to labels : Optional<Map<String,String>> in method public SingularityDeploy(requestId String, id String, command Optional<String>, arguments Optional<List<String>>, containerInfo Optional<SingularityContainerInfo>, customExecutorCmd Optional<String>, customExecutorId Optional<String>, customExecutorSource Optional<String>, customExecutorResources Optional<Resources>, customExecutorUser Optional<String>, resources Optional<Resources>, env Optional<Map<String,String>>, taskEnv Optional<Map<Integer,Map<String,String>>>, uris Optional<List<String>>, metadata Optional<Map<String,String>>, executorData Optional<ExecutorData>, version Optional<String>, timestamp Optional<Long>, labels Optional<Map<String,String>>, taskLabels Optional<Map<Integer,Map<String,String>>>, deployHealthTimeoutSeconds Optional<Long>, healthcheckUri Optional<String>, healthcheckIntervalSeconds Optional<Long>, healthcheckTimeoutSeconds Optional<Long>, healthcheckPortIndex Optional<Integer>, healthcheckMaxRetries Optional<Integer>, healthcheckMaxTotalTimeoutSeconds Optional<Long>, serviceBasePath Optional<String>, loadBalancerGroups Optional<Set<String>>, loadBalancerPortIndex Optional<Integer>, considerHealthyAfterRunningForSeconds Optional<Long>, loadBalancerOptions Optional<Map<String,Object>>, loadBalancerDomains Optional<Set<String>>, loadBalancerAdditionalRoutes Optional<List<String>>, loadBalancerTemplate Optional<String>, skipHealthchecksOnDeploy Optional<Boolean>, healthcheckProtocol Optional<HealthcheckProtocol>, deployInstanceCountPerStep Optional<Integer>, deployStepWaitTimeMs Optional<Integer>, autoAdvanceDeploySteps Optional<Boolean>, maxTaskRetries Optional<Integer>, shell Optional<Boolean>) in class com.hubspot.singularity.SingularityDeploy | From | To | |
Change Parameter Type parameters : Optional<SingularityDockerParameters> to parameters : Optional<Map<String,String>> in method public SingularityDockerInfo(image String, privileged boolean, network SingularityDockerNetworkType, portMappings Optional<List<SingularityDockerPortMapping>>, forcePullImage Optional<Boolean>, parameters Optional<Map<String,String>>) in class com.hubspot.mesos.SingularityDockerInfo | From | To | |
Change Parameter Type labels : Optional<SingularityMesosTaskLabels> to labels : Optional<Map<String,String>> in method public setLabels(labels Optional<Map<String,String>>) : SingularityDeployBuilder in class com.hubspot.singularity.SingularityDeployBuilder | From | To | |
Rename Variable | 1 | ||
Rename Variable dockerParameter : SingularityDockerParameter to entry : Map.Entry<String,String> in method private prepareContainerInfo(offer Offer, taskId SingularityTaskId, bldr TaskInfo.Builder, containerInfo SingularityContainerInfo, ports Optional<long[]>) : void in class com.hubspot.singularity.mesos.SingularityMesosTaskBuilder | From | To | |
Change Return Type | 3 | ||
Change Return Type Optional<SingularityMesosTaskLabels> to Optional<Map<String,String>> in method public getLabels() : Optional<Map<String,String>> in class com.hubspot.singularity.SingularityDeployBuilder | From | To | |
Change Return Type Optional<SingularityDockerParameters> to Map<String,String> in method public getParameters() : Map<String,String> in class com.hubspot.mesos.SingularityDockerInfo | From | To | |
Change Return Type Optional<SingularityMesosTaskLabels> to Optional<Map<String,String>> in method public getLabels() : Optional<Map<String,String>> in class com.hubspot.singularity.SingularityDeploy | From | To | |
Change Variable Type | 1 | ||
Change Variable Type dockerParameter : SingularityDockerParameter to entry : Map.Entry<String,String> in method private prepareContainerInfo(offer Offer, taskId SingularityTaskId, bldr TaskInfo.Builder, containerInfo SingularityContainerInfo, ports Optional<long[]>) : void in class com.hubspot.singularity.mesos.SingularityMesosTaskBuilder | From | To | |
Change Attribute Type | 3 | ||
Change Attribute Type parameters : Optional<SingularityDockerParameters> to parameters : Map<String,String> in class com.hubspot.mesos.SingularityDockerInfo | From | To | |
Change Attribute Type labels : Optional<SingularityMesosTaskLabels> to labels : Optional<Map<String,String>> in class com.hubspot.singularity.SingularityDeploy | From | To | |
Change Attribute Type labels : Optional<SingularityMesosTaskLabels> to labels : Optional<Map<String,String>> in class com.hubspot.singularity.SingularityDeployBuilder | From | To |