Skip to content

Update ETA example to include timezone#1

Merged
amantri merged 2 commits intomainfrom
docs-eta
Sep 18, 2023
Merged

Update ETA example to include timezone#1
amantri merged 2 commits intomainfrom
docs-eta

Conversation

@amantri
Copy link
Owner

@amantri amantri commented Sep 16, 2023

Description

The behavior when using timezone unaware datetime objects can be incorrect. Setting the timezone removes any ambiguity, especially in a distributed environment.

For a Redis broker running on the same machine as my Celery queue, I got incorrect scheduling when using naive datetime objects. Setting the timezone explicitly fixed the issue.

The behavior when using timezone unaware datetime objects can be incorrect.

For a Redis broker running on the same machine as my Celery queue, I got incorrect scheduling when using naive datetime objects. Setting the timezone explicitly fixed the issue.
@amantri amantri merged commit 07dcb44 into main Sep 18, 2023
@amantri amantri deleted the docs-eta branch September 18, 2023 18:57
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.

1 participant