[
https://issues.apache.org/jira/browse/IMPALA-14629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18053157#comment-18053157
]
ASF subversion and git services commented on IMPALA-14629:
----------------------------------------------------------
Commit 4963ee6d2272669a103c7cca98abd3b4353b5016 in impala's branch
refs/heads/master from Csaba Ringhofer
[ https://gitbox.apache.org/repos/asf?p=impala.git;h=4963ee6d2 ]
IMPALA-14629: Implement st_point(double,double) in c++
Adds native implementation for the common 2d constructor while
keeps the more complex st_point(string) in Java. Tried using
boost's WKT parser but making it work with >2d geometries
seemed non-trivial.
Also removes the following overloads:
st_point(double,double,double)
st_point(double,double,double,double)
st_pointz(double,double,double,double)
These conflict with overloads that have different semantics in
PostGis, see HIVE-29395 for details.
Testing:
- no test changes are needed - the remaining constructors
are covered while the removed ones were not used at all
Change-Id: I927413f92cf4d4e9a995f7024de0ec2e3b584b6d
Reviewed-on: http://gerrit.cloudera.org:8080/23874
Reviewed-by: Impala Public Jenkins <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Implement st_point() in c++
> ---------------------------
>
> Key: IMPALA-14629
> URL: https://issues.apache.org/jira/browse/IMPALA-14629
> Project: IMPALA
> Issue Type: Sub-task
> Components: Backend
> Reporter: Csaba Ringhofer
> Priority: Major
>
> st_point has several overloads, including one that accepts STRING parameter
> interpreted as WKT ( boost::geometry's WKT parser could be used).
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
