Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
SWJTU-ZhangLei closed pull request #39372: [fix](cloud) Fix `TabletIndexPB` missing db_id URL: https://github.com/apache/doris/pull/39372 -- 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

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295671542 TPC-H: Total hot run time: 38419 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295593432 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295572764 ClickBench: Total hot run time: 31.71 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295568530 TPC-DS: Total hot run time: 195384 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295558820 TPC-H: Total hot run time: 38407 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2295494090 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
SWJTU-ZhangLei commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1721114208 ## fe/fe-core/src/main/java/org/apache/doris/cloud/datasource/CloudInternalCatalog.java: ## @@ -180,6 +180,7 @@ protected Partition createPartitionWithIndices(lo

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-18 Thread via GitHub
SWJTU-ZhangLei commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1721114049 ## cloud/src/recycler/recycler.cpp: ## @@ -2644,4 +2645,165 @@ bool InstanceRecycler::check_recycle_tasks() { return found; } +int InstanceRecycler::repa

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
gavinchou commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1720138868 ## cloud/src/recycler/recycler.cpp: ## @@ -592,7 +592,8 @@ int InstanceRecycler::do_recycle() { .add(task_wrapper([this]() { return InstanceRecycler

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
gavinchou commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1720132627 ## cloud/src/recycler/recycler.cpp: ## @@ -2644,4 +2645,165 @@ bool InstanceRecycler::check_recycle_tasks() { return found; } +int InstanceRecycler::repair_ta

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2293407936 ClickBench: Total hot run time: 31.56 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2293398181 TPC-DS: Total hot run time: 191392 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2293380597 TPC-H: Total hot run time: 38708 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-16 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2293288279 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290990879 ClickBench: Total hot run time: 30.89 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290983865 TPC-DS: Total hot run time: 190861 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290968005 TPC-H: Total hot run time: 38629 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290915833 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
github-actions[bot] commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290847521 PR approved by anyone and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the UR

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-15 Thread via GitHub
github-actions[bot] commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290847486 PR approved by at least one committer and no changes requested. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHu

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
github-actions[bot] commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1718035453 ## cloud/src/meta-service/meta_service.cpp: ## @@ -403,9 +403,9 @@ void MetaServiceImpl::batch_get_version(::google::protobuf::RpcController* contr }

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
github-actions[bot] commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1718033733 ## cloud/src/meta-service/meta_service.cpp: ## @@ -403,9 +403,9 @@ void MetaServiceImpl::batch_get_version(::google::protobuf::RpcController* contr }

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1718030465 ## cloud/src/recycler/recycler.cpp: ## @@ -2640,4 +2641,165 @@ bool InstanceRecycler::check_recycle_tasks() { return found; } +int InstanceRecycler::repa

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290766049 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1718028622 ## cloud/src/meta-service/meta_service.cpp: ## @@ -406,7 +406,7 @@ void MetaServiceImpl::batch_get_version(::google::protobuf::RpcController* contr void intern

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1718028771 ## cloud/src/meta-service/meta_service.cpp: ## @@ -618,11 +618,13 @@ void MetaServiceImpl::create_tablets(::google::protobuf::RpcController* controll m

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290505320 TPC-H: Total hot run time: 37719 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
w41ter commented on code in PR #39372: URL: https://github.com/apache/doris/pull/39372#discussion_r1717823383 ## cloud/src/recycler/recycler.cpp: ## @@ -2640,4 +2641,165 @@ bool InstanceRecycler::check_recycle_tasks() { return found; } +int InstanceRecycler::repair_table

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290443311 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2290379345 run buildall -- 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.

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
doris-robot commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2288738997 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+process+y

[PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei opened a new pull request, #39372: URL: https://github.com/apache/doris/pull/39372 * In cloud mode, when creating tablet missing record `db_id` field in `TabletIndexPB`, so add a background task in recycler to fill `db_id` in `TabletIndexPB` ## Proposed changes

Re: [PR] [fix](cloud) Fix `TabletIndexPB` missing db_id [doris]

2024-08-14 Thread via GitHub
SWJTU-ZhangLei commented on PR #39372: URL: https://github.com/apache/doris/pull/39372#issuecomment-2288739234 run buildall -- 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.