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

Disable AOSP K/V Call log backups #249

Closed
wants to merge 2 commits into from
Closed

Disable AOSP K/V Call log backups #249

wants to merge 2 commits into from

Conversation

Uldiniad
Copy link
Contributor

@Uldiniad Uldiniad commented May 7, 2021

K/V Backup of call logs creates one pair per call log entry, thus
leading to thousands of files for people with lots of logs.
This is really slow especially with Nextcloud, making the backup take hours.
Fix that by not enabling the AOSP K/V code, and instead using our own
full backup code.

Full Backup implementation: https://review.calyxos.org/c/CalyxOS/platform_packages_providers_ContactsProvider/+/2970

Issue: calyxos#395
Issue: #127
Issue: #224
Change-Id: I742408ab62e126129966f7c9d095d6e61fdc58f6

K/V Backup of call logs creates one pair per call log entry, thus
leading to thousands of files for people with lots of logs.
This is really slow especially with Nextcloud, making the backup take hours.
Fix that by not enabling the AOSP K/V code, and instead using our own
full backup code.

Full Backup implementation: https://review.calyxos.org/c/CalyxOS/platform_packages_providers_ContactsProvider/+/2970

Issue: calyxos#395
Issue: #127
Change-Id: I742408ab62e126129966f7c9d095d6e61fdc58f6
@chirayudesai chirayudesai requested a review from grote May 7, 2021 14:01
@chirayudesai
Copy link
Member

@grote one thing I noticed with this change is, if you have an old backup with the K/V backup, and then upgrade to a build with this which makes a full backup - restore still restores both backups.

I was thinking of disabling allowBackup for the AOSP CallLogProvider, would that help with restore too?

If not, maybe we need to add some code here.

@chirayudesai
Copy link
Member

Won't be needed after #127 (comment)

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