Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
Gabriel39 merged PR #27641: URL: https://github.com/apache/doris/pull/27641 -- 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.ap

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
github-actions[bot] commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1831129925 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] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829414271 run pipelinex_p0 -- 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

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
doris-robot commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829378147 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 1ea6db7e9a4d17761389cac68b70b7f96fbb5875, data reload: false

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
doris-robot commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829364520 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 47.9 seconds stream load tsv: 574 seconds loaded 74807831229 Bytes, a

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829336296 run p0 -- 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 unsubscrib

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-28 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829336512 run pipelinex_p0 -- 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

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
github-actions[bot] commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1407305556 ## be/src/pipeline/pipeline_fragment_context.h: ## @@ -222,10 +226,18 @@ class PipelineFragmentContext : public std::enable_shared_from_this _dml_query_sta

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1829216958 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. To unsu

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
HappenLee commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1407044654 ## be/src/vec/exec/vexchange_node.cpp: ## @@ -147,11 +147,17 @@ void VExchangeNode::release_resource(RuntimeState* state) { Status VExchangeNode::collect_query_st

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
HappenLee commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1407043756 ## be/src/runtime/query_statistics.h: ## @@ -155,5 +158,12 @@ class QueryStatisticsRecvr { std::map _query_statistics; SpinLock _lock; }; +// QueryStatisti

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
github-actions[bot] commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1828952035 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] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
github-actions[bot] commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1828952003 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] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
doris-robot commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1827721466 (From new machine)TeamCity pipeline, clickbench performance test result: the sum of best hot time: 46.76 seconds stream load tsv: 564 seconds loaded 74807831229 Bytes,

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
doris-robot commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1827714833 TPC-H test result on machine: 'aliyun_ecs.c7a.8xlarge_32C64G' ``` Tpch sf100 test result on commit 4c87e54853b0f33189e9845d639aa67992cd66de, data reload: false

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
github-actions[bot] commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1406009395 ## be/src/runtime/query_statistics.h: ## @@ -17,6 +17,7 @@ #pragma once +#include Review Comment: warning: 'gen_cpp/FrontendService_types.h' file

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1827620079 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. To unsu

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
yiguolei commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1405987620 ## be/src/pipeline/pipeline_fragment_context.cpp: ## @@ -132,6 +132,7 @@ PipelineFragmentContext::PipelineFragmentContext( _task_group_entity = _query_ctx->ge

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
yiguolei commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1405987030 ## be/src/vec/exec/vexchange_node.cpp: ## @@ -147,11 +147,17 @@ void VExchangeNode::release_resource(RuntimeState* state) { Status VExchangeNode::collect_query_sta

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
github-actions[bot] commented on code in PR #27641: URL: https://github.com/apache/doris/pull/27641#discussion_r1405963481 ## be/src/runtime/query_statistics.h: ## @@ -107,11 +107,13 @@ class QueryStatistics { } void to_pb(PQueryStatistics* statistics); - +void t

Re: [PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
Mryange commented on PR #27641: URL: https://github.com/apache/doris/pull/27641#issuecomment-1827567270 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. To unsu

[PR] [feature](audit-log) add audit-log in insert into [doris]

2023-11-27 Thread via GitHub
Mryange opened a new pull request, #27641: URL: https://github.com/apache/doris/pull/27641 ## Proposed changes 1. In the past, audit-log used the result node for sending, while insert did not. Therefore, we notify fe during the send report for this. 2. Fixed a bug where information