[CP] fix st_intersects/st_covers performance

obdev · ob-robot · commit fc805e458d6c · 2026-01-11T04:09:37.000Z
diff --git a/src/sql/engine/expr/ob_expr_st_covers.cpp b/src/sql/engine/expr/ob_expr_st_covers.cpp
@@ -153,11 +153,6 @@ int ObExprPrivSTCovers::eval_st_covers_common(const ObExpr &expr, ObEvalCtx &ctx
     ObGeoEvalCtx gis_context(*mem_ctx, srs);
     bool result = false;
     if (OB_FAIL(ret)) {
-    } else if (OB_FAIL(gis_context.append_geo_arg(geo2)) || OB_FAIL(gis_context.append_geo_arg(geo1))) {
-      LOG_WARN("build gis context failed", K(ret), K(gis_context.get_geo_count()));
-    } else if (OB_FAIL(ObGeoFunc<ObGeoFuncType::CoveredBy>::geo_func::eval(gis_context, result))) {
-      LOG_WARN("eval st coveredBy failed", K(ret));
-      ObGeoExprUtils::geo_func_error_handle(ret, N_PRIV_ST_COVERS);
     } else {
       ObGeoEvalCtx gis_context(*mem_ctx);
       bool result = false;
diff --git a/src/sql/engine/expr/ob_expr_st_intersects.cpp b/src/sql/engine/expr/ob_expr_st_intersects.cpp
@@ -148,23 +148,6 @@ int ObExprSTIntersects::eval_st_intersects(const ObExpr &expr, ObEvalCtx &ctx, O
           LOG_WARN("add geo2 to const cache failed", K(ret));
         }
       }
-      ObGeoEvalCtx gis_context(*mem_ctx);
-      bool result = false;
-      if (OB_FAIL(ret)) {
-      } else if (OB_FAIL(gis_context.append_geo_arg(geo1)) || OB_FAIL(gis_context.append_geo_arg(geo2))) {
-        LOG_WARN("build gis context failed", K(ret), K(gis_context.get_geo_count()));
-      } else if (OB_FAIL(ObGeoFunc<ObGeoFuncType::Intersects>::geo_func::eval(gis_context, result))) {
-        LOG_WARN("eval st intersection failed", K(ret));
-        ObGeoExprUtils::geo_func_error_handle(ret, N_ST_INTERSECTS);
-      } else {
-        if (geo1->type() == ObGeoType::POINT
-            && geo2->type() == ObGeoType::POINT
-            && result == true
-            && OB_FAIL(ObGeoTypeUtil::eval_point_box_intersects(srs, geo1, geo2, result))) {
-          LOG_WARN("eval box intersection failed", K(ret));
-        }
-      }
-
       if (OB_FAIL(ret)) {
       } else if (OB_FAIL(ObGeoExprUtils::get_intersects_res(*geo1, *geo2, gis_arg1, gis_arg2, 
                                                             const_param_cache, srs, 
