Csaba Ringhofer has uploaded this change for review. ( http://gerrit.cloudera.org:8080/23874
Change subject: IMPALA-14629: Implement st_point(double,double) in c++ ...................................................................... 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 overlaods 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 --- M be/src/exprs/geo/geospatial-functions-ir.cc M be/src/exprs/geo/geospatial-functions.h M be/src/exprs/geo/shape-format.h M fe/src/compat-hive-3/java/org/apache/impala/compat/HiveEsriGeospatialBuiltins.java 4 files changed, 54 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/74/23874/1 -- To view, visit http://gerrit.cloudera.org:8080/23874 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I927413f92cf4d4e9a995f7024de0ec2e3b584b6d Gerrit-Change-Number: 23874 Gerrit-PatchSet: 1 Gerrit-Owner: Csaba Ringhofer <[email protected]>
