[
https://issues.apache.org/jira/browse/IMPALA-14576?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Work on IMPALA-14576 started by Csaba Ringhofer.
------------------------------------------------
> Add st_EnvIntersects before other st_ relations
> ------------------------------------------------
>
> Key: IMPALA-14576
> URL: https://issues.apache.org/jira/browse/IMPALA-14576
> Project: IMPALA
> Issue Type: Sub-task
> Components: Frontend
> Reporter: Csaba Ringhofer
> Assignee: Csaba Ringhofer
> Priority: Major
>
> Example rewrite:
> {code}
> st_intersects(<geom1>, <geom2>) ->
> st_envintersects(<geom1>, <geom2>) AND st_intersects(<geom1>, <geom2>)
> {code}
> This can't be implemented as a simple expression rewrite rule because the
> original st_intersects() is unchanged and the same rule could be applied
> again. A new "inferred predicate" logic is needed that applies a rule only
> once to the expression tree.
> This may be a temporary but significant improvement - st_envintersects() is
> much easier to port to c++ than the "full" relation functions like
> st_intersects.
> st_envIntersects is rewritten here:
> https://gerrit.cloudera.org/#/c/23700/6/be/src/exprs/geo/geospatial-functions-ir.cc
> other st_ relations are rewritten here using boost::geometry:
> https://gerrit.cloudera.org/#/c/20602/6/be/src/exprs/geo/relation-wrapper.h
> The first change is much simpler and safer and could be merged in the near
> future.
> Note that IMPALA-14571 ensures that native functions are evaluated before
> Java ones in predicates, so st_envIntersects would be evaluated before the
> "full" relation (if c++ rewrite for st_envIntersects gets merged while other
> relations stay in Java).
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
