fix(sqlalchemy): add support for sqlalchemy string subclassed types

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

@@ -304,6 +304,8 @@ def to_ibis(cls, typ: sat.TypeEngine, nullable: bool = True) -> dt.DataType:
         elif isinstance(typ, sa.DateTime):
             timezone = "UTC" if typ.timezone else None
             return dt.Timestamp(timezone, nullable=nullable)
+        elif isinstance(typ, sat.String):
+            return dt.String(nullable=nullable)
         elif geospatial_supported and isinstance(typ, ga.types._GISType):
             name = typ.geometry_type.upper()
             try:

ibis/backends/tests/test_client.py

@@ -210,6 +210,11 @@ def tmpcon(alchemy_con):
     return alchemy_con._from_url(alchemy_con.con.url)
 
 
+@mark.broken(
+    ["oracle"],
+    reason="oracle temp tables aren't cleaned up on reconnect -- they need to "
+    "be switched from using atexit to weakref.finalize",
+)
 @mark.notimpl(["trino", "druid"], reason="doesn't implement temporary tables")
 @mark.notyet(
     ["sqlite"], reason="sqlite only support temporary tables in temporary databases"