Csaba Ringhofer has posted comments on this change. ( http://gerrit.cloudera.org:8080/20602 )
Change subject: WIP: port critical geospatial functions to c++ ...................................................................... Patch Set 6: Started taking this patch apart and commit pieces: https://gerrit.cloudera.org/#/c/23700/ First PS includes only commit that use the header of geometries but not have to process the whole format. The reason is that the current patch is practically untouched since years, it would be a lot of effort to clean it up. There are also question marks in some case about the usefulness of the design, e.g. for WKT parsing. More benchmarks are needed to compare the speed of the Java vs c++ implementations for geometries of different complexity. -- To view, visit http://gerrit.cloudera.org:8080/20602 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I692ebc13617c37d61f77fffb09e872095a0fd11c Gerrit-Change-Number: 20602 Gerrit-PatchSet: 6 Gerrit-Owner: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Daniel Becker <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Comment-Date: Fri, 21 Nov 2025 16:33:18 +0000 Gerrit-HasComments: No
