[
https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16534282#comment-16534282
]
ASF subversion and git services commented on GEODE-5355:
--------------------------------------------------------
Commit bb85e7c32a9cba9717fa22907997ca3c3bdab28a in geode's branch
refs/heads/develop from [[email protected]]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=bb85e7c ]
GEODE-5355: Remove BackupUtil and BackupDataStoreHelper
* Rename *Operation classes to *Step
* Introduce new BackupOperation as internal API for launching backup
> Refactor backup code to unify it as one API and make it testable
> ----------------------------------------------------------------
>
> Key: GEODE-5355
> URL: https://issues.apache.org/jira/browse/GEODE-5355
> Project: Geode
> Issue Type: Improvement
> Components: persistence
> Reporter: Kirk Lund
> Assignee: Kirk Lund
> Priority: Major
> Labels: pull-request-available
> Time Spent: 40m
> Remaining Estimate: 0h
>
> BackupUtil and BackupDataStoreHelper are collections of static methods that
> aren't easy to test with mocks.
> JMX, GFSH and the deprecated Admin API treat BackupUtil and
> BackupDataStoreHelper as entry points to the Backup API whereas BackupService
> is presented to the User as the only API. We should have one class
> (BackupService) be the single entry point to performing a Backup.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
