Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24648 )
Change subject: IMPALA-15193: [addendum] ST_Disjoint Incorrect Results Workaround ...................................................................... IMPALA-15193: [addendum] ST_Disjoint Incorrect Results Workaround Workaround of a bug in the ST_Disjoint built-in UDF caused by a bug in the underpinning esri-geometry-api library. When comparing two geometries to determine if they are disjoint where the first geometry is either a LineString or a MultiLineString and the second geometry is a MultiLineString, then ST_Disjoint incorrectly returns false when the first geometry is within the envelope of the second. The first patch for this issue fixed the ST_Intersects function. Since the underlying bug is in the disjoint predicate implementation of the esri-geometry-api library, the workaround must also be applied to the ST_Disjoint function implementation. Tested via the addition of new query tests with the test results double checked in postgis. Assisted-by: Github Copilot (GPT-5.3-Codex) Change-Id: I66c23c6db1140add91d86dbaeaec9884bcb6a831 Reviewed-on: http://gerrit.cloudera.org:8080/24648 Reviewed-by: Csaba Ringhofer <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M java/hive-geospatial-functions/src/main/java/org/apache/impala/hive/geospatial/esri/ST_Disjoint.java M testdata/workloads/functional-query/queries/QueryTest/geospatial-esri-extra.test 2 files changed, 70 insertions(+), 1 deletion(-) Approvals: Csaba Ringhofer: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/24648 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I66c23c6db1140add91d86dbaeaec9884bcb6a831 Gerrit-Change-Number: 24648 Gerrit-PatchSet: 2 Gerrit-Owner: Jason Fehr <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Jason Fehr <[email protected]>
