Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
zhangstar333 merged PR #47265: URL: https://github.com/apache/doris/pull/47265 -- 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

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
Mryange commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2608691442 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
github-actions[bot] commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2608689985 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
github-actions[bot] commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2608689941 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2607160858 TeamCity be ut coverage result: Function Coverage: 41.82% (10902/26070) Line Coverage: 32.09% (91951/286539) Region Coverage: 31.24% (47123/150848) Branch Coverage: 27.

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2607145755 ClickBench: Total hot run time: 30.46 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2607134290 TPC-DS: Total hot run time: 195242 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2607108459 TPC-H: Total hot run time: 32403 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-22 Thread via GitHub
Mryange commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2606939093 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
BiteThet commented on code in PR #47265: URL: https://github.com/apache/doris/pull/47265#discussion_r1924761325 ## be/src/common/mock/mock_runtime_state.h: ## @@ -0,0 +1,35 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agre

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
BiteThet commented on code in PR #47265: URL: https://github.com/apache/doris/pull/47265#discussion_r1924762876 ## be/test/vec/exec/sort/sort_test.cpp: ## @@ -0,0 +1,183 @@ +// Licensed to the Apache Software Foundation (ASF) under one +// or more contributor license agreeme

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
BiteThet commented on code in PR #47265: URL: https://github.com/apache/doris/pull/47265#discussion_r1924762214 ## be/src/runtime/descriptors.h: ## @@ -53,7 +54,7 @@ class PSlotDescriptor; class SlotDescriptor { public: -// virtual ~SlotDescriptor() {}; +MOCK_DEF

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2604491290 TeamCity be ut coverage result: Function Coverage: 41.84% (10909/26071) Line Coverage: 32.11% (92005/286519) Region Coverage: 31.25% (47137/150837) Branch Coverage: 27.

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
Mryange commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2604365378 run beut -- 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 unsubscr

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2604108734 ClickBench: Total hot run time: 30.55 s ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/clickbench-tools

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2604089840 TPC-DS: Total hot run time: 189635 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpcds-tools TPC-DS

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-21 Thread via GitHub
doris-robot commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2604015152 TPC-H: Total hot run time: 33006 ms ``` machine: 'aliyun_ecs.c7a.8xlarge_32C64G' scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools Tpch sf100

Re: [PR] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-20 Thread via GitHub
Mryange commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2603871843 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-20 Thread via GitHub
Thearas commented on PR #47265: URL: https://github.com/apache/doris/pull/47265#issuecomment-2603812684 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] [Test] (BE-UT)Add sorter BE-UT and foundational mocks [doris]

2025-01-20 Thread via GitHub
Mryange opened a new pull request, #47265: URL: https://github.com/apache/doris/pull/47265 ### What problem does this PR solve? 1. Add BEUT tests for the sorter. 2. Introduce some mock basic types for future development. 3. Introduce BE_BENCHMARK because some functions are only defin