Skip to content

Commit

Permalink
fix(druid): ensure that string types are translated to VARCHAR
Browse files Browse the repository at this point in the history
  • Loading branch information
cpcloud authored and jcrist committed Sep 12, 2023
1 parent 1d1f7bd commit 56e6ffc
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
5 changes: 5 additions & 0 deletions ibis/backends/druid/datatypes.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,11 @@ def _smallint(element, compiler, **kw):
return "SMALLINT"


@compiles(DruidString, "druid")
def _string(element, compiler, **kw):
return "VARCHAR"


class DruidType(AlchemyType):
dialect = "hive"

Expand Down
1 change: 0 additions & 1 deletion ibis/backends/tests/test_string.py
Original file line number Diff line number Diff line change
Expand Up @@ -1059,7 +1059,6 @@ def test_levenshtein(con, right):
reason="doesn't allow boolean expressions in select statements",
raises=sa.exc.OperationalError,
)
@pytest.mark.notyet(["druid"], raises=sa.exc.ProgrammingError)
@pytest.mark.broken(
["oracle"],
reason="sqlalchemy converts True to 1, which cannot be used in CASE WHEN statement",
Expand Down

0 comments on commit 56e6ffc

Please sign in to comment.