Skip to content
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

Replace pendulum with arrow #5069

Closed
karolzlot opened this issue Oct 19, 2021 · 3 comments
Closed

Replace pendulum with arrow #5069

karolzlot opened this issue Oct 19, 2021 · 3 comments
Labels
enhancement An improvement of an existing feature

Comments

@karolzlot
Copy link

karolzlot commented Oct 19, 2021

Current behavior

Currently prefect uses pendulum module.

pendulum was created because at the time arrow module had many bugs and pendulum addressed them. Currently all those bugs are fixed in arrow: source

pendulum currently isn't very actively developed compared to arrow.

Proposed behavior

Replace pendulum with arrow

@karolzlot karolzlot added the enhancement An improvement of an existing feature label Oct 19, 2021
@zanieb
Copy link
Contributor

zanieb commented Oct 19, 2021

Hey @karolzlot -- we use pendulum across our entire stack and haven't encountered any issues. I understand arrow is being developed more actively lately, but I'm not sure that motivates a change. Is there a way this would change your experience?

@karolzlot
Copy link
Author

No, it won't change.

I think this may be good idea to change it in long-term, not asap.

@zanieb
Copy link
Contributor

zanieb commented Oct 19, 2021

Thanks for bringing it to our attention but we do not have plans to switch now. We'll keep arrow in mind if we have issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

2 participants