csringhofer commented on code in PR #6581: URL: https://github.com/apache/hive/pull/6581#discussion_r3638070183
########## ql/src/java/org/apache/hadoop/hive/ql/udf/esri/ST_GeodesicLengthWGS84.java: ########## @@ -17,14 +17,13 @@ */ package org.apache.hadoop.hive.ql.udf.esri; Review Comment: Impala's handling of this changed recently, I copied the whole dir from Hive to Impala to be able to make modifications: https://github.com/apache/impala/commit/d51c3c12fbfa1cbfe348b64f9e6006b3f45f9b56 So from Impala's point of view Hive is free to modify / rename / delete these functions. My plan is to keep the original ESRI lib around for a while to provide compatibility for old workloads, and deprecate it at same point when the new lib seems stable. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
