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.

Sign up for GitHub

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

Jump to bottom

Simpler send batches command #186

Closed

lullis wants to merge 1 commit into WikiMovimentoBrasil:main from lullis:simpler_send_batches

Collaborator

lullis commented Feb 21, 2025

Calling lock_batch is not needed, given that batch.run already implements the state change at its very first step
Why create a method (send_start_message()) that just adds another layer of indirection to a logging function?
get_first_batch can be avoided by using queryset iterators.


          Simpler send batches command

caea7f2

lullis closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet