bchapuis commented on code in PR #2988: URL: https://github.com/apache/calcite/pull/2988#discussion_r1034723743
########## core/src/main/java/org/apache/calcite/runtime/ProjectionTransformer.java: ########## @@ -40,6 +40,14 @@ /** * Transforms the projection of a geometry. + * + * This class uses Proj4J to transform the projection of a geometry + * and should not be used beyond the scope of the Spatial Type Extensions. + * Proj4J is released under the Apache License 2.0, however, it also uses the EPSG dataset, + * which has restricting <a href="https://epsg.org/terms-of-use.html";>terms of use</a>. + * As a result, Proj4J is not suitable for inclusion in Apache Calcite + * and this class will throw {@code ClassNotFoundException}s + * if Proj4J is not added to the classpath by the user. Review Comment: This is a good idea, I added notices in `reference.md` and `spatial.md`. -- 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]
