fix(sqlalchemy): handle generic geography/geometry by name instead of geotype

Author: cpcloud

ibis/backends/base/sql/alchemy/datatypes.py

@@ -296,7 +296,7 @@ def sa_json(_, satype, nullable=True):
 
     @dt.dtype.register(Dialect, (ga.Geometry, ga.types._GISType))
     def ga_geometry(_, gatype, nullable=True):
-        t = gatype.geometry_type
+        t = gatype.geometry_type.upper()
         if t == 'POINT':
             return dt.Point(nullable=nullable)
         if t == 'LINESTRING':
@@ -309,8 +309,8 @@ def ga_geometry(_, gatype, nullable=True):
             return dt.MultiPoint(nullable=nullable)
         if t == 'MULTIPOLYGON':
             return dt.MultiPolygon(nullable=nullable)
-        if t == 'GEOMETRY':
-            return dt.Geometry(nullable=nullable)
+        if t in ('GEOMETRY', 'GEOGRAPHY'):
+            return getattr(dt, gatype.name.lower())(nullable=nullable)
         else:
             raise ValueError(f"Unrecognized geometry type: {t}")