acelyc111 commented on code in PR #1139:
URL: https://github.com/apache/incubator-pegasus/pull/1139#discussion_r961540213
##########
src/rdsn/src/replica/backup/replica_backup_manager.h:
##########
@@ -102,8 +134,13 @@ class replica_backup_manager : replica_base
backup_status::type _status{backup_status::UNINITIALIZED};
int64_t _backup_id{0};
error_code _checkpoint_err{ERR_OK};
+ error_code _upload_err{ERR_IO_PENDING};
Review Comment:
Is it OK to initialize it to 'ERR_OK'?
##########
src/rdsn/src/replica/backup/replica_backup_manager.h:
##########
@@ -59,14 +59,40 @@ class replica_backup_manager : replica_base
private:
void try_to_checkpoint(const int64_t &backup_id, /*out*/ backup_response
&response);
+ void try_to_upload(const std::string &provider_type,
+ const std::string &root_path,
+ const std::string app_name,
Review Comment:
`const std::string &app_name`?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@pegasus.apache.org
For additional commands, e-mail: dev-h...@pegasus.apache.org
