-
Notifications
You must be signed in to change notification settings - Fork 289
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(python): add docstring for limit behavior #800
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approve barring the change about limit == 0
*WARNING* if you have a large dataset, removing | ||
the limit can potentially result in reading a | ||
large amount of data into memory and cause | ||
out of memory issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Even if I call to_batches
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we don't have to_batches
api yet. I'll create a follow-up issue
python/lancedb/query.py
Outdated
|
||
Returns | ||
------- | ||
LanceQueryBuilder | ||
The LanceQueryBuilder object. | ||
""" | ||
self._limit = limit | ||
if limit is None or limit < 0: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What if limit == 0
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
made limit == 0 also act as if there's no limit
Closes #796