Skip to content

Commit

Permalink
feat(snowflake): implement proper approx median
Browse files Browse the repository at this point in the history
  • Loading branch information
cpcloud authored and kszucs committed Apr 25, 2023
1 parent 41bacf2 commit b15a6fe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions ibis/backends/snowflake/registry.py
Original file line number Diff line number Diff line change
Expand Up @@ -389,6 +389,7 @@ def _group_concat(t, op):
ops.EndsWith: fixed_arity(sa.func.endswith, 2),
ops.GroupConcat: _group_concat,
ops.Hash: unary(sa.func.hash),
ops.ApproxMedian: reduction(lambda x: sa.func.approx_percentile(x, 0.5)),
}
)

Expand Down
2 changes: 1 addition & 1 deletion ibis/backends/tests/test_aggregation.py
Original file line number Diff line number Diff line change
Expand Up @@ -1010,7 +1010,7 @@ def test_corr_cov(


@pytest.mark.notimpl(
["datafusion", "mysql", "sqlite", "snowflake", "mssql", "druid"],
["datafusion", "mysql", "sqlite", "mssql", "druid"],
raises=com.OperationNotDefinedError,
)
@pytest.mark.broken(
Expand Down

0 comments on commit b15a6fe

Please sign in to comment.