2e52789edf68016e7a3f450164f8bd3d8e6cb210
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 6
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Parameter Type | 1 | ||
| Change Parameter Type appId : ApplicationId to appPath : String in method private loadApplicationAttemptState(appState ApplicationStateData, appPath String) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Rename Variable | 3 | ||
| Rename Variable childNodePath : String to parentNodePath : String in method private loadRMAppState(rmState RMState) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Rename Variable nodeCreatePath : String to path : String in method private handleApplicationAttemptStateOp(appAttemptId ApplicationAttemptId, attemptStateDataPB ApplicationAttemptStateData, operation AppAttemptOp) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Rename Variable childData : byte[] to appData : byte[] in method private loadRMAppStateFromAppNode(rmState RMState, appNodePath String, appIdStr String) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Extract Variable | 1 | ||
| Extract Variable appIdPath : String in method public appExists(app RMApp) : boolean from class org.apache.hadoop.yarn.server.resourcemanager.recovery.TestZKRMStateStore.TestZKRMStateStoreTester | From | To | |
| Extract Method | 5 | ||
| Extract Method private createStore(conf Configuration) : RMStateStore extracted from public getRMStateStore() : RMStateStore in class org.apache.hadoop.yarn.server.resourcemanager.recovery.TestZKRMStateStore.TestZKRMStateStoreTester | From | To | |
| Extract Method private loadRMAppStateFromAppNode(rmState RMState, appNodePath String, appIdStr String) : void extracted from private loadRMAppState(rmState RMState) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Extract Method private handleApplicationAttemptStateOp(appAttemptId ApplicationAttemptId, attemptStateDataPB ApplicationAttemptStateData, operation AppAttemptOp) : void extracted from protected removeApplicationAttemptInternal(appAttemptId ApplicationAttemptId) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Extract Method private handleApplicationAttemptStateOp(appAttemptId ApplicationAttemptId, attemptStateDataPB ApplicationAttemptStateData, operation AppAttemptOp) : void extracted from protected storeApplicationAttemptStateInternal(appAttemptId ApplicationAttemptId, attemptStateDataPB ApplicationAttemptStateData) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Extract Method private removeApp(removeAppId String, safeRemove boolean, attempts Set<ApplicationAttemptId>) : void extracted from protected removeApplicationStateInternal(appState ApplicationStateData) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To | |
| Rename Parameter | 1 | ||
| Rename Parameter appId : ApplicationId to appPath : String in method private loadApplicationAttemptState(appState ApplicationStateData, appPath String) : void in class org.apache.hadoop.yarn.server.resourcemanager.recovery.ZKRMStateStore | From | To |