SWJTU-ZhangLei commented on code in PR #53075:
URL: https://github.com/apache/doris/pull/53075#discussion_r2203770864
##########
be/src/olap/rowset/rowset_meta.cpp:
##########
@@ -112,16 +113,20 @@ Result<const StorageResource*>
RowsetMeta::remote_storage_resource() {
}
if (!_storage_resource.fs) {
- // not initialized yet
- auto storage_resource = get_storage_resource(resource_id());
- if (storage_resource) {
+ if (auto storage_resource = get_storage_resource(resource_id())) {
_storage_resource = std::move(storage_resource->first);
} else {
+ if (config::is_cloud_mode()) {
+
ExecEnv::GetInstance()->storage_engine().to_cloud().sync_storage_vault();
Review Comment:
> 加点注释 为什么会出现这种需要临时sync的情况, 另外 这个会不会比启动的时候第一次sync要更快到达 导致 启动那一次 sync
其实可能并不是第一次.
1、启动的时候可能心跳服务还没启动，可能出现sync没有成功，这个时候查询的请求已经发到be了，可能出现这种还未sync的情况；
2、新建的存储资源，也可能出现还没同步的情况，这个时候查询来了是不是就出问题了
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
