[
https://issues.apache.org/jira/browse/IMPALA-14571?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18040384#comment-18040384
]
ASF subversion and git services commented on IMPALA-14571:
----------------------------------------------------------
Commit f6ceca2b4d37d6a6d6f3881ddb3509165d1deb2a in impala's branch
refs/heads/master from Csaba Ringhofer
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=f6ceca2b4 ]
IMPALA-14571: increase planner cost of java functions
The main motivation is to evaluate expensive geospatial
functions (which are Java functions) last in predicates.
Java functions have a major overhead anyway from the JNI
call, so bumping all Java function costs seems beneficial.
Note that currently geospatial functions are the only
built-in Java functions.
Change-Id: I11d1652d76092ec60af18a33502dacc25b284fcc
Reviewed-on: http://gerrit.cloudera.org:8080/22733
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Increase "cost" of geospatial functions during planning
> -------------------------------------------------------
>
> Key: IMPALA-14571
> URL: https://issues.apache.org/jira/browse/IMPALA-14571
> Project: IMPALA
> Issue Type: Sub-task
> Components: Frontend
> Reporter: Csaba Ringhofer
> Assignee: Csaba Ringhofer
> Priority: Major
>
> Example:
> select * from functional.binary_tbl
> where st_geomfromwkb(binary_col) is not null and sqrt(id) = 1;
> The planner considers the cost of both function calls's cost '10' and keeps
> evaluating st_geomfromwkb() first:
> https://github.com/apache/impala/blob/3ce0004c12f587dbc6ac6d7451b5f362d148d033/fe/src/main/java/org/apache/impala/analysis/Expr.java#L91
> This is generally a bad decision, as st_geomfromwkb() contains both a
> native->JVM call and the processing of WKB binary format.
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
