Skip to content

Commit

Permalink
fix(dask/pandas): allow limit to be None
Browse files Browse the repository at this point in the history
  • Loading branch information
cpcloud authored and kszucs committed Jun 16, 2022
1 parent 84b2bea commit 9f91d6b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion ibis/backends/dask/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ def execute(
limit: str = 'default',
**kwargs,
):
if limit != 'default':
if limit != 'default' and limit is not None:
raise ValueError(
'limit parameter to execute is not yet implemented in the '
'dask backend'
Expand Down
2 changes: 1 addition & 1 deletion ibis/backends/pandas/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ class Backend(BasePandasBackend):
def execute(self, query, params=None, limit='default', **kwargs):
from ibis.backends.pandas.core import execute_and_reset

if limit != 'default':
if limit != 'default' and limit is not None:
raise ValueError(
'limit parameter to execute is not yet implemented in the '
'pandas backend'
Expand Down

0 comments on commit 9f91d6b

Please sign in to comment.