Skip to content

Include only the last 90 days of downloads in our database dumps #3611

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

Merged

Conversation

jtgeibel
Copy link
Member

In #3479 we plan to drop old entries and archive them in some other way, so old entries will eventually disappear from dumps anyway. This should make use of the database dumps much more practical for daily use. I think it would be reasonable to even limit this to the past week of data.

r? @pietroalbini
cc @kornelski, #2078

@kornelski
Copy link
Contributor

It's OK for me personally, because I have a copy of the older data. But how will others get the full downloads history?

Do you plan to implement dated archives for downloads like this?

@pietroalbini
Copy link
Member

Do you plan to implement dated archives for downloads like this?

@kornelski yes, see #3479

@pietroalbini
Copy link
Member

@bors r+

Thanks!

@bors
Copy link
Contributor

bors commented May 14, 2021

📌 Commit c38a5f5 has been approved by pietroalbini

@bors
Copy link
Contributor

bors commented May 14, 2021

⌛ Testing commit c38a5f5 with merge 99ef346...

@bors
Copy link
Contributor

bors commented May 14, 2021

☀️ Test successful - checks-actions
Approved by: pietroalbini
Pushing 99ef346 to master...

@bors bors merged commit 99ef346 into rust-lang:master May 14, 2021
@pietroalbini
Copy link
Member

@kornelski fyi we opened a locked issue tracking all the changes happening to the database dump you can subscribe to, so we can avoid manually pinging you when we think a change might impact you: #3617

@kornelski
Copy link
Contributor

Thanks

@jtgeibel jtgeibel deleted the filter-db-dump-to-90-days-of-downloads branch May 14, 2021 15:48
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.

6 participants