Skip to content

Cancelation actions#13

Open
DamiCassinotti wants to merge 2 commits into
masterfrom
cancelation
Open

Cancelation actions#13
DamiCassinotti wants to merge 2 commits into
masterfrom
cancelation

Conversation

@DamiCassinotti

Copy link
Copy Markdown
Contributor

No description provided.

print(f"Found a contab job to delete ${job}")
cron.remove(job)

def cancel_operation(self, operation):

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ojo con esto, esto se llama si el cliente tiene que reanudar un Cron porque se cayó o cuando inicia una actividad, por lo que nunca van a haber TASKs sin finalizar => finish operation siempre va a finalizar la ejecución y eliminar el scheduler, ahora, si justo te cae un cancel_operation del servidor (DELETE a /delete), es probable que te queden TASKS enviandose al servidor y todavía se vayan a enviar, esto no es tan malo pero te van a llegar nuevos resultados por más que hayas cancelado, supongo que es el comportamiento deseado

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No veo mal que se sigan enviando mediciones después del /delete, total si ya están los resultados como que no perdemos nada en mandarlas, quizás para algo sirvan

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants