dcbcb94fd2695edd4bbca63b2759292e99695d47
Files Diff
- Added: 2
- Removed: 1
- Renamed: 0
- Modified: 24
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Rename Method | 1 | ||
| Rename Method public loadRemoteFunctions(version long) : boolean renamed to public syncWithRemoteRegistry(version long) : boolean in class org.apache.drill.exec.expr.fn.FunctionImplementationRegistry | From | To | |
| Inline Method | 2 | ||
| Inline Method private findDrillFunction(functionResolver FunctionResolver, functionCall FunctionCall, retry boolean) : DrillFuncHolder inlined to public findDrillFunction(functionResolver FunctionResolver, functionCall FunctionCall) : DrillFuncHolder in class org.apache.drill.exec.expr.fn.FunctionImplementationRegistry | From | To | |
| Inline Method private getPhysicalPlan(handler AbstractSqlHandler, sqlNode SqlNode, context QueryContext) : PhysicalPlan inlined to public getPlan(context QueryContext, sql String, textPlan Pointer<String>) : PhysicalPlan in class org.apache.drill.exec.planner.sql.DrillSqlWorker | From | To | |
| Change Return Type | 2 | ||
| Change Return Type boolean to void in method private removeAllByJar(jarName String) : void in class org.apache.drill.exec.expr.fn.registry.FunctionRegistryHolder | From | To | |
| Change Return Type long to void in method public setFunctionRegistryVersion(version long) : void in class org.apache.drill.exec.planner.sql.DrillOperatorTable | From | To | |
| Extract Variable | 4 | ||
| Extract Variable stat : Stat in method public hasPath(path String, consistent boolean, version DataChangeVersion) : boolean from class org.apache.drill.exec.coord.zk.ZookeeperClient | From | To | |
| Extract Variable updatedFunctions : List<DrillFuncHolder> in method public findDrillFunction(functionResolver FunctionResolver, functionCall FunctionCall) : DrillFuncHolder from class org.apache.drill.exec.expr.fn.FunctionImplementationRegistry | From | To | |
| Extract Variable functions : List<DrillFuncHolder> in method public findDrillFunction(functionResolver FunctionResolver, functionCall FunctionCall) : DrillFuncHolder from class org.apache.drill.exec.expr.fn.FunctionImplementationRegistry | From | To | |
| Extract Variable expectedVersion : long in method public testVersion() : void from class org.apache.drill.exec.expr.fn.registry.FunctionRegistryHolderTest | From | To | |
| Change Attribute Type | 1 | ||
| Change Attribute Type version : AtomicInteger to version : int in class org.apache.drill.exec.testing.store.NoWriteLocalStore | From | To | |
| Inline Variable | 1 | ||
| Inline Variable functionRegistry : FunctionImplementationRegistry in method private getPhysicalPlan(handler AbstractSqlHandler, sqlNode SqlNode, context QueryContext) : PhysicalPlan from class org.apache.drill.exec.planner.sql.DrillSqlWorker | From | To | |
| Extract Method | 4 | ||
| Extract Method private getQueryPlan(context QueryContext, sql String, textPlan Pointer<String>) : PhysicalPlan extracted from public getPlan(context QueryContext, sql String, textPlan Pointer<String>) : PhysicalPlan in class org.apache.drill.exec.planner.sql.DrillSqlWorker | From | To | |
| Extract Method public put(key String, value V, dataChangeVersion DataChangeVersion) : void extracted from public put(key String, value V) : void in class org.apache.drill.exec.store.sys.store.LocalPersistentStore | From | To | |
| Extract Method public get(key String, dataChangeVersion DataChangeVersion) : V extracted from public get(key String) : V in class org.apache.drill.exec.store.sys.store.LocalPersistentStore | From | To | |
| Extract Method public hasPath(path String, consistent boolean, version DataChangeVersion) : boolean extracted from public hasPath(path String, consistent boolean) : boolean in class org.apache.drill.exec.coord.zk.ZookeeperClient | From | To |