Skip to content

Commit 2c7bc89

Browse files
uros-dbcloud-fan
authored andcommitted
[SPARK-53760][GEO][SQL][FOLLOWUP] Fix error message and comment for Spatial types
### What changes were proposed in this pull request? - Fix the `ST_INVALID_SRID_VALUE` error message. - Fix the `SpatialType` doc comment. ### Why are the changes needed? Followup PR to clean up original issues introduced in: #52491. ### Does this PR introduce _any_ user-facing change? Yes, the `ST_INVALID_SRID_VALUE` error message is updated. ### How was this patch tested? Existing tests are sufficient. ### Was this patch authored or co-authored using generative AI tooling? No. Closes #52632 from uros-db/geo-logical-types-FOLLOWUP. Authored-by: Uros Bojanic <uros.bojanic@databricks.com> Signed-off-by: Wenchen Fan <wenchen@databricks.com>
1 parent 1c0bca9 commit 2c7bc89

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

common/utils/src/main/resources/error/error-conditions.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5573,7 +5573,7 @@
55735573
},
55745574
"ST_INVALID_SRID_VALUE" : {
55755575
"message" : [
5576-
"Invalid or unsupported SRID (spatial reference identifier) value: <srid>"
5576+
"Invalid or unsupported SRID (spatial reference identifier) value: <srid>."
55775577
],
55785578
"sqlState" : "22023"
55795579
},

sql/api/src/main/scala/org/apache/spark/sql/types/SpatialType.scala

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ import org.apache.spark.sql.types.AbstractDataType
2222
trait SpatialType extends AbstractDataType {
2323

2424
/**
25-
* Mixed SRID value and the corresponding CRS for geospatial types (Geometry and Geography)
25+
* Mixed SRID value and the corresponding CRS for geospatial types (Geometry and Geography).
2626
* These values represent a geospatial type that can hold different SRID values per row.
2727
*/
2828
final val MIXED_SRID: Int = -1

0 commit comments

Comments
 (0)