Arnab Karmakar has posted comments on this change. ( http://gerrit.cloudera.org:8080/23604 )
Change subject: IMPALA-13263: Add single-argument overload for ST_ConvexHull() and tests ...................................................................... Patch Set 2: (4 comments) http://gerrit.cloudera.org:8080/#/c/23604/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/23604/1//COMMIT_MSG@9 PS1, Line 9: T_ConvexHull() to : support addi > Can you explain that here with an example? Done http://gerrit.cloudera.org:8080/#/c/23604/1//COMMIT_MSG@11 PS1, Line 11: o be calle > nit: geometric Done http://gerrit.cloudera.org:8080/#/c/23604/1//COMMIT_MSG@12 PS1, Line 12: representations without explicit conversion. : > Adding a Testing header and mention the name and nature of the tests. Done http://gerrit.cloudera.org:8080/#/c/23604/1/testdata/workloads/functional-query/queries/QueryTest/geospatial-esri.test File testdata/workloads/functional-query/queries/QueryTest/geospatial-esri.test: http://gerrit.cloudera.org:8080/#/c/23604/1/testdata/workloads/functional-query/queries/QueryTest/geospatial-esri.test@2720 PS1, Line 2720: ConvexHull(BINARY) overl > Can we add some negative tests to see if all cases are handled? Also add a Done -- To view, visit http://gerrit.cloudera.org:8080/23604 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Idb17d98f5e75929ec0143aa16195a84dd6e50796 Gerrit-Change-Number: 23604 Gerrit-PatchSet: 2 Gerrit-Owner: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Arnab Karmakar <[email protected]> Gerrit-Reviewer: Csaba Ringhofer <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Pranav Lodha <[email protected]> Gerrit-Comment-Date: Tue, 28 Oct 2025 10:45:07 +0000 Gerrit-HasComments: Yes
