[
https://issues.apache.org/jira/browse/GEODE-5405?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Kirk Lund updated GEODE-5405:
-----------------------------
Description:
* FlushToDiskBackupStep should extend (abstract) BackupStep. This will remove
some duplication.
* BackupService has a non-private field which should be encapsulated
(currentTask).
* validateRequestingAdmin should be renamed to validateRequestingSender in
BackupService.
* Cleanup and streamline integration and distributed tests in backup package.
* Introduce new backup config that doesn't use constants and properties.
was:
* FlushToDiskBackupStep should extend (abstract) BackupStep. This will remove
some duplication.
* BackupService has a non-private field which should be encapsulated
(currentTask).
* validateRequestingAdmin should be renamed to validateRequestingSender in
BackupService.
> Refactor backup code to remove duplication and improve understanding
> --------------------------------------------------------------------
>
> Key: GEODE-5405
> URL: https://issues.apache.org/jira/browse/GEODE-5405
> Project: Geode
> Issue Type: Improvement
> Components: persistence
> Reporter: Kirk Lund
> Assignee: Kirk Lund
> Priority: Major
> Labels: pull-request-available
> Time Spent: 10m
> Remaining Estimate: 0h
>
> * FlushToDiskBackupStep should extend (abstract) BackupStep. This will remove
> some duplication.
> * BackupService has a non-private field which should be encapsulated
> (currentTask).
> * validateRequestingAdmin should be renamed to validateRequestingSender in
> BackupService.
> * Cleanup and streamline integration and distributed tests in backup package.
> * Introduce new backup config that doesn't use constants and properties.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
