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

SOLR-15101: Add list/delete APIs for incremental backups #2336

Merged
merged 12 commits into from
Feb 15, 2021

Conversation

gerlowskija
Copy link
Contributor

Description

SOLR-13608 introduced support into Solr for an "incremental" backup file structure, which allows storing multiple backup points for the same collection at a given location. With the ability to store multiple backups at the same place, users will need to be able to list and cleanup these backups.

Solution

This PR introduces two new APIs: one for listing the backups at a given location (along with associated metadata), and one to delete or cleanup these backups. The APIs are offered in both v1 and v2 flavors.

Tests

Manual testing, along with new automated tests in PurgeGraphTest (reference checking for detecting index files to delete), V2CollectionBackupsAPIMappingTest (v1<->v2 mapping), and AbstractIncrementalBackupTest (integration test for list, delete functionality).

Checklist

Please review the following and check all that apply:

  • I have reviewed the guidelines for How to Contribute and my code conforms to the standards described there to the best of my ability.
  • I have created a Jira issue and added the issue ID to my pull request title.
  • I have given Solr maintainers access to contribute to my PR branch. (optional but recommended)
  • I have developed this patch against the master branch.
  • I have run ./gradlew check.
  • I have added tests for my changes.
  • I have added documentation for the Ref Guide (for Solr changes only).

Jason Gerlowski added 12 commits February 8, 2021 19:43
SOLR-13608 introduces a new "incremental" backup format, which allows
storage of multiple backup "points" in the same location. This
development introduces a need for APIs to manage these potentially
plural backups.

This commit introduces /admin/collections?action=LISTBACKUPS and
/admin/collections?action=DELETEBACKUP to handle these backups.
@gerlowskija gerlowskija merged commit f993c39 into apache:master Feb 15, 2021
@gerlowskija gerlowskija deleted the LUCENESOLR_2013_only_pr branch February 15, 2021 12:04
gerlowskija added a commit to gerlowskija/lucene-solr-1 that referenced this pull request Feb 16, 2021
SOLR-13608 introduces a new "incremental" backup format, which allows
storage of multiple backup "points" in the same location. This
development introduces a need for APIs to manage these potentially
plural backups.

This commit introduces /admin/collections?action=LISTBACKUPS and
/admin/collections?action=DELETEBACKUP to handle these backups.
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