You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
rest.delete_messages should allow the use of both sync and async/lazy iterators.
Why is this needed?
Users should not be forced to chunk iterators themselves if they want to delete and fetch the history at the same time.
Ideal implementation
Instead of having a list of pending messages, it would be better to cast the iterable into a LazyIterator and use iterator.chunk(). While this may reduce the performance by a few nanoseconds it will improve user experience.
The text was updated successfully, but these errors were encountered:
Summary
rest.delete_messages
should allow the use of both sync and async/lazy iterators.Why is this needed?
Users should not be forced to chunk iterators themselves if they want to delete and fetch the history at the same time.
Ideal implementation
Instead of having a list of pending messages, it would be better to cast the iterable into a LazyIterator and use
iterator.chunk()
. While this may reduce the performance by a few nanoseconds it will improve user experience.The text was updated successfully, but these errors were encountered: