github-actions[bot] commented on code in PR #16813: URL: https://github.com/apache/doris/pull/16813#discussion_r1122935636
########## be/src/olap/olap_define.h: ########## @@ -98,6 +98,17 @@ static const std::string PENDING_DELTA_PREFIX = "pending_delta"; static const std::string INCREMENTAL_DELTA_PREFIX = "incremental_delta"; static const std::string CLONE_PREFIX = "clone"; +// define paths +static inline std::string remote_tablet_path(int64_t tablet_id) { + // data/{tablet_id} + return fmt::format("{}/{}", DATA_PREFIX, tablet_id); +} +static inline std::string remote_tablet_meta_path(int64_t tablet_id, int64_t replica_id, + int64_t cooldown_term) { + // data/{tablet_id}/{replica_id}.{cooldown_term}.meta + return fmt::format("{}/{}.{}.meta", remote_tablet_path(tablet_id), replica_id, cooldown_term); +} + static const std::string TABLET_UID = "tablet_uid"; static const std::string STORAGE_NAME = "storage_name"; Review Comment: warning: unused function 'remote_tablet_meta_path' [clang-diagnostic-unused-function] ```cpp A_PREFIX, tablet_id); ^ ``` -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org