[
https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16535431#comment-16535431
]
ASF subversion and git services commented on GEODE-5355:
--------------------------------------------------------
Commit 935743586a874dce4b4b780aefe0c653d42f8902 in geode's branch
refs/heads/feature/GEODE-QueryProvider from [[email protected]]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=9357435 ]
GEODE-5355: Fix apache license header of new backup classes
> 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
> Fix For: 1.7.0
>
> 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)
