Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-19 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1961690658 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-19 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1961689022 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-19 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1961686380 ## fe/fe-core/src/main/java/org/apache/doris/catalog/Index.java: ## @@ -268,10 +268,20 @@ public OlapFile.TabletIndexPB toPb(Map columnMap) { OlapFile.Tab

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959786396 ## fe/fe-core/src/main/java/org/apache/doris/backup/RestoreJob.java: ## @@ -981,14 +983,25 @@ private void checkAndPrepareMeta() { LOG.debug("finished

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959784671 ## fe/fe-core/src/main/java/org/apache/doris/backup/RestoreJob.java: ## @@ -972,6 +970,10 @@ private void checkAndPrepareMeta() { db.readUnlock();

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959782547 ## fe/fe-core/src/main/java/org/apache/doris/backup/RestoreJob.java: ## @@ -742,6 +745,13 @@ private void checkAndPrepareMeta() { Parti

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959765393 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959757181 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959738796 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-18 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1959737716 ## fe/fe-core/src/main/java/org/apache/doris/cloud/backup/CloudRestoreJob.java: ## @@ -0,0 +1,649 @@ +// Licensed to the Apache Software Foundation (ASF) under one

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957286027 ## fe/fe-core/src/main/java/org/apache/doris/catalog/OlapTable.java: ## @@ -873,8 +875,14 @@ && getTableProperty().getDynamicPartitionProperty().getBuckets()

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957285627 ## be/src/cloud/cloud_snapshot_loader.cpp: ## @@ -0,0 +1,234 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements.

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957285627 ## be/src/cloud/cloud_snapshot_loader.cpp: ## @@ -0,0 +1,234 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements.

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957285627 ## be/src/cloud/cloud_snapshot_loader.cpp: ## @@ -0,0 +1,234 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreements.

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957283735 ## fe/fe-core/src/main/java/org/apache/doris/catalog/PartitionInfo.java: ## @@ -465,13 +465,15 @@ public void readFields(DataInput in) throws IOException { }

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957282702 ## fe/fe-core/src/main/java/org/apache/doris/catalog/Partition.java: ## @@ -350,6 +351,11 @@ public boolean visualiseShadowIndex(long shadowIndexId, boolean isBaseIndex)

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-16 Thread via GitHub
xy720 commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1957282702 ## fe/fe-core/src/main/java/org/apache/doris/catalog/Partition.java: ## @@ -350,6 +351,11 @@ public boolean visualiseShadowIndex(long shadowIndexId, boolean isBaseIndex)

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956193363 ## fe/fe-core/src/main/java/org/apache/doris/catalog/PartitionInfo.java: ## @@ -465,13 +465,15 @@ public void readFields(DataInput in) throws IOException {

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956192657 ## fe/fe-core/src/main/java/org/apache/doris/catalog/Partition.java: ## @@ -350,6 +351,11 @@ public boolean visualiseShadowIndex(long shadowIndexId, boolean isBase

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956190868 ## fe/fe-core/src/main/java/org/apache/doris/catalog/OlapTable.java: ## @@ -873,8 +875,14 @@ && getTableProperty().getDynamicPartitionProperty().getBuckets()

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956183031 ## be/src/cloud/cloud_snapshot_loader.cpp: ## @@ -0,0 +1,234 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agree

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956181777 ## be/src/cloud/cloud_snapshot_loader.cpp: ## @@ -0,0 +1,234 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agree

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-02-14 Thread via GitHub
dataroaring commented on code in PR #47300: URL: https://github.com/apache/doris/pull/47300#discussion_r1956168762 ## gensrc/thrift/AgentService.thrift: ## @@ -372,6 +372,8 @@ struct TDownloadReq { 5: optional Types.TStorageBackendType storage_backend = Types.TStorageBacke

[PR] [draft](restore) Add cloud restore job [doris]

2025-01-21 Thread via GitHub
xy720 opened a new pull request, #47300: URL: https://github.com/apache/doris/pull/47300 ### What problem does this PR solve? Issue Number: close #xxx Related PR: #xxx Problem Summary: design: https://github.com/user-attachments/assets/64edd594-b8ed-49e0-9ee

Re: [PR] [draft](restore) Add cloud restore job [doris]

2025-01-21 Thread via GitHub
hello-stephen commented on PR #47300: URL: https://github.com/apache/doris/pull/47300#issuecomment-2605265356 Thank you for your contribution to Apache Doris. Don't know what should be done next? See [How to process your PR](https://cwiki.apache.org/confluence/display/DORIS/How+to+pro