38bc20940de061de72bde7b6298b8c65e17e07ec
Files Diff
- Added: 2
- Removed: 1
- Renamed: 0
- Modified: 10
Refactorings reported:
| Refactoring Name | Occurences | ||
|---|---|---|---|
| Move Method | 5 | ||
| Move Method private newRadiusSymbol(context Context, expression Expression) : Optional<Symbol> from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private newRadiusSymbol(context Context, expression Expression) : Optional<Symbol> from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Move Method private addProjection(context Context, node PlanNode, symbol Symbol, expression Expression) : PlanNode from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private addProjection(context Context, node PlanNode, symbol Symbol, expression Expression) : PlanNode from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Move Method private toExpression(optionalSymbol Optional<Symbol>, defaultExpression Expression) : Expression from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private toExpression(optionalSymbol Optional<Symbol>, defaultExpression Expression) : Expression from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Move Method private checkAlignment(joinNode JoinNode, maybeLeftSymbols Set<Symbol>, maybeRightSymbols Set<Symbol>) : int from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private checkAlignment(joinNode JoinNode, maybeLeftSymbols Set<Symbol>, maybeRightSymbols Set<Symbol>) : int from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Move Method private containsNone(values Collection<Symbol>, testValues Collection<Symbol>) : boolean from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private containsNone(values Collection<Symbol>, testValues Collection<Symbol>) : boolean from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Move Attribute | 1 | ||
| Move Attribute private GEOMETRY_TYPE_SIGNATURE : TypeSignature from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin to private GEOMETRY_TYPE_SIGNATURE : TypeSignature from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates | From | To | |
| Extract Class | 1 | ||
| Extract Class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates from class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin | From | To | |
| Move Class | 1 | ||
| Move Class com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicateToJoin moved to com.facebook.presto.sql.planner.iterative.rule.TransformSpatialPredicates.TransformSpatialPredicateToJoin | From | To | |
| Extract Method | 4 | ||
| Extract Method private assertSpatialJoin(taskContext TaskContext, joinType Type, buildPages RowPagesBuilder, probePages RowPagesBuilder, expected MaterializedResult) : void extracted from public testSpatialJoin() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoinOperator | From | To | |
| Extract Method private assertSpatialJoin(taskContext TaskContext, joinType Type, buildPages RowPagesBuilder, probePages RowPagesBuilder, expected MaterializedResult) : void extracted from public testEmptyProbe() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoinOperator | From | To | |
| Extract Method private appendProbe() : void extracted from private processProbe() : void in class com.facebook.presto.operator.SpatialJoinOperator | From | To | |
| Extract Method private assertSpatialJoin(taskContext TaskContext, joinType Type, buildPages RowPagesBuilder, probePages RowPagesBuilder, expected MaterializedResult) : void extracted from public testEmptyBuild() : void in class com.facebook.presto.plugin.geospatial.TestSpatialJoinOperator | From | To |