Hello Balazs Hevele, Impala Public Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24551
to look at the new patch set (#4).
Change subject: POC IMPALA-15166: Improve test coverage for geospatial functions
......................................................................
POC IMPALA-15166: Improve test coverage for geospatial functions
Add more function test cases to geospatial-esri-extra.test and
geospatial-esri-srid.test to cover all st_ functions.
Add the geospatial-wkb-serialization.test (WKB round-trip and malformed
input cases) to the default HIVE_ESRI test run, and the
functional.geom_relations_wkt test table (WKT strings in a text table)
with its data file. The table is used by the geospatial relation predicate
tests to check all geometry type combinations for all relations. Getting
data from tables also allow checking things like executing with codegen,
which will be important for the c++ implementation.
Change-Id: Ic155e691cdb93842502e52c0a09807dd08e4a4f3
Assisted-by: Claude Opus 4.8
---
A testdata/data/geom_relations_wkt/data.csv
M testdata/datasets/functional/functional_schema_template.sql
M testdata/datasets/functional/schema_constraints.csv
M
testdata/workloads/functional-query/queries/QueryTest/geospatial-esri-extra.test
M
testdata/workloads/functional-query/queries/QueryTest/geospatial-esri-srid.test
A
testdata/workloads/functional-query/queries/QueryTest/geospatial-relations-table.test
A
testdata/workloads/functional-query/queries/QueryTest/geospatial-wkb-serialization.test
M tests/custom_cluster/test_geospatial_library.py
M tests/query_test/test_geospatial_functions.py
9 files changed, 1,267 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/51/24551/4
--
To view, visit http://gerrit.cloudera.org:8080/24551
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic155e691cdb93842502e52c0a09807dd08e4a4f3
Gerrit-Change-Number: 24551
Gerrit-PatchSet: 4
Gerrit-Owner: Csaba Ringhofer <[email protected]>
Gerrit-Reviewer: Balazs Hevele <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>