f70b0cd2d5f8cd666a58f46de012931730616060
Files Diff
- Added: 0
- Removed: 0
- Renamed: 0
- Modified: 10
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Change Variable Type | 1 | ||
| Change Variable Type probeChannel : Integer to probeChannel : int in method private createSpatialLookupJoin(node SpatialJoinNode, probeNode PlanNode, probeSource PhysicalOperation, probeSymbol Symbol, pagesSpatialIndexFactory PagesSpatialIndexFactory, context LocalExecutionPlanContext) : OperatorFactory in class com.facebook.presto.sql.planner.LocalExecutionPlanner.Visitor | From | To | |
| Extract Variable | 1 | ||
| Extract Variable probeChannelGetter : Function<Symbol,Integer> in method private createSpatialLookupJoin(node SpatialJoinNode, probeNode PlanNode, probeSource PhysicalOperation, probeSymbol Symbol, pagesSpatialIndexFactory PagesSpatialIndexFactory, context LocalExecutionPlanContext) : OperatorFactory from class com.facebook.presto.sql.planner.LocalExecutionPlanner.Visitor | From | To | |
| Extract Method | 4 | ||
| Extract Method private testSpatialJoinIntersects(session Session) : void extracted from public testBroadcastSpatialJoinIntersects() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoins | From | To | |
| Extract Method private testSpatialJoinContains(session Session) : void extracted from public testBroadcastSpatialJoinContains() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoins | From | To | |
| Extract Method private testDistanceQuery(session Session) : void extracted from public testBroadcastDistanceQuery() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoins | From | To | |
| Extract Method private buildIndex(driverContext DriverContext, spatialRelationshipTest SpatialPredicate, radiusChannel Optional<Integer>, partitionChannel Optional<Integer>, kdbTreeJson Optional<String>, filterFunction Optional<InternalJoinFilterFunction>, buildPages RowPagesBuilder) : PagesSpatialIndexFactory extracted from private buildIndex(driverContext DriverContext, spatialRelationshipTest SpatialPredicate, radiusChannel Optional<Integer>, filterFunction Optional<InternalJoinFilterFunction>, buildPages RowPagesBuilder) : PagesSpatialIndexFactory in class com.facebook.presto.plugin.geospatial.TestSpatialJoinOperator | From | To |