[jira] [Commented] (GEODE-5355) Refactor backup code to unify it as one API and make it testable
[ https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16535430#comment-16535430 ] ASF subversion and git services commented on GEODE-5355: Commit bb85e7c32a9cba9717fa22907997ca3c3bdab28a in geode's branch refs/heads/feature/GEODE-QueryProvider from [~apa...@the9muses.net] [ 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 > 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)
[jira] [Commented] (GEODE-5355) Refactor backup code to unify it as one API and make it testable
[ https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16535431#comment-16535431 ] ASF subversion and git services commented on GEODE-5355: Commit 935743586a874dce4b4b780aefe0c653d42f8902 in geode's branch refs/heads/feature/GEODE-QueryProvider from [~apa...@the9muses.net] [ 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)
[jira] [Commented] (GEODE-5355) Refactor backup code to unify it as one API and make it testable
[ https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16534285#comment-16534285 ] ASF subversion and git services commented on GEODE-5355: Commit 935743586a874dce4b4b780aefe0c653d42f8902 in geode's branch refs/heads/develop from [~apa...@the9muses.net] [ 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 > 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)
[jira] [Commented] (GEODE-5355) Refactor backup code to unify it as one API and make it testable
[ https://issues.apache.org/jira/browse/GEODE-5355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16534282#comment-16534282 ] ASF subversion and git services commented on GEODE-5355: Commit bb85e7c32a9cba9717fa22907997ca3c3bdab28a in geode's branch refs/heads/develop from [~apa...@the9muses.net] [ 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)