-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
cache=True
(default) in to_datetime
causes too big slow-down
#15736
Labels
P-medium
Priority: medium
performance
Performance issues or improvements
python
Related to Python Polars
Comments
MarcoGorelli
added
bug
Something isn't working
python
Related to Python Polars
needs triage
Awaiting prioritization by a maintainer
performance
Performance issues or improvements
and removed
bug
Something isn't working
labels
Apr 18, 2024
I think |
As we discussed, we should use |
reswqa
added
P-medium
Priority: medium
and removed
needs triage
Awaiting prioritization by a maintainer
labels
Apr 19, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
P-medium
Priority: medium
performance
Performance issues or improvements
python
Related to Python Polars
Checks
Reproducible example
Log output
No response
Issue description
I don't think the
cache
should hurt performance - I get the desire to speed it up in the case of duplicates, but unique datetimes is a pretty common case tooIs it possible to limit the cache so that it only holds, say, 50 elements? And then, any new element ejects an existing one from the cache?
Expected behavior
Similar performance
Installed versions
The text was updated successfully, but these errors were encountered: