check for future geometry versions, add test for future compatability

Author: Maxxen

spatial/include/spatial/core/geometry/geometry_processor.hpp

@@ -220,8 +220,13 @@ class GeometryProcessor {
 public:
 	RESULT Process(const geometry_t &geom, ARGS... args) {
 
-		has_z = geom.GetProperties().HasZ();
-		has_m = geom.GetProperties().HasM();
+		const auto props = geom.GetProperties();
+
+		// Check the version
+		props.CheckVersion();
+
+		has_z = props.HasZ();
+		has_m = props.HasM();
 		nesting_level = 0;
 		current_type = geom.GetType();
 		parent_type = GeometryType::POINT;

spatial/include/spatial/core/geometry/geometry_properties.hpp

@@ -5,6 +5,8 @@ namespace spatial {
 
 namespace core {
 
+static constexpr const uint8_t GEOMETRY_VERSION = 0;
+
 struct GeometryProperties {
 private:
 	static constexpr const uint8_t Z = 0x01;
@@ -14,6 +16,8 @@ struct GeometryProperties {
 	// static constexpr const uint8_t EMPTY = 0x08;
 	// static constexpr const uint8_t GEODETIC = 0x10;
 	// static constexpr const uint8_t SOLID = 0x20;
+	static constexpr const uint8_t VERSION_1 = 0x40;
+	static constexpr const uint8_t VERSION_0 = 0x80;
 	uint8_t flags = 0;
 
 public:
@@ -24,6 +28,16 @@ struct GeometryProperties {
 		SetM(has_m);
 	}
 
+	inline void CheckVersion() const {
+		const auto v0 = (flags & VERSION_0);
+		const auto v1 = (flags & VERSION_1);
+		if ((v1 | v0) != GEOMETRY_VERSION) {
+			throw NotImplementedException(
+			    "This geometry seems to be written with a newer version of the DuckDB spatial library that is not "
+			    "compatible with this version. Please upgrade your DuckDB installation.");
+		}
+	}
+
 	inline bool HasZ() const {
 		return (flags & Z) != 0;
 	}

spatial/include/spatial/core/geometry/geometry_type.hpp

@@ -83,13 +83,15 @@ class geometry_t {
 	}
 
 	GeometryType GetType() const {
+		// return the type
 		return Load<GeometryType>(const_data_ptr_cast(data.GetPrefix()));
 	}
+
 	GeometryProperties GetProperties() const {
-		return Load<GeometryProperties>(const_data_ptr_cast(data.GetPrefix() + 1));
-	}
-	uint16_t GetHash() const {
-		return Load<uint16_t>(const_data_ptr_cast(data.GetPrefix() + 2));
+		const auto props = Load<GeometryProperties>(const_data_ptr_cast(data.GetPrefix() + 1));
+		// Check the version
+		props.CheckVersion();
+		return props;
 	}
 
 	bool TryGetCachedBounds(Box2D<double> &bbox) const {
@@ -101,6 +103,9 @@ class geometry_t {
 		auto hash = cursor.Read<uint16_t>();
 		(void)hash;
 
+		// Check the version
+		properties.CheckVersion();
+
 		if (properties.HasBBox()) {
 			cursor.Skip(4); // skip padding