You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@doris.apache.org by "feiniaofeiafei (via GitHub)" <gi...@apache.org> on 2024/03/15 08:17:08 UTC

[PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

feiniaofeiafei opened a new pull request, #32290:
URL: https://github.com/apache/doris/pull/32290

   db reported a error " expression duplicate in grouping set" when there are duplicate expression in grouping set.
   e.g.select a from mal_test1 group by grouping sets((a,a))
   This pr removes duplicate expr in grouping set :
   select a from mal_test1 group by grouping sets((a))


-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "feiniaofeiafei (via GitHub)" <gi...@apache.org>.
feiniaofeiafei commented on PR #32290:
URL: https://github.com/apache/doris/pull/32290#issuecomment-1999142564

   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 unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #32290:
URL: https://github.com/apache/doris/pull/32290#issuecomment-1999145913

   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
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "924060929 (via GitHub)" <gi...@apache.org>.
924060929 merged PR #32290:
URL: https://github.com/apache/doris/pull/32290


-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #32290:
URL: https://github.com/apache/doris/pull/32290#issuecomment-1999142335

   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+your+PR)


-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "doris-robot (via GitHub)" <gi...@apache.org>.
doris-robot commented on PR #32290:
URL: https://github.com/apache/doris/pull/32290#issuecomment-1999173306

   
   <details>
   <summary>TPC-H: <b>Total hot run time: 38466 ms</b></summary>
   
   ```
   machine: 'aliyun_ecs.c7a.8xlarge_32C64G'
   scripts: https://github.com/apache/doris/tree/master/tools/tpch-tools
   Tpch sf100 test result on commit bccc8bfd8b1138b1fbdf8970d5a8415fac742b65, data reload: false
   
   ------ Round 1 ----------------------------------
   q1	17655	4438	4110	4110
   q2	2026	150	152	150
   q3	10685	1127	898	898
   q4	7463	710	734	710
   q5	7470	2608	2639	2608
   q6	185	123	122	122
   q7	1203	823	815	815
   q8	9332	2045	2019	2019
   q9	7125	6471	6406	6406
   q10	8444	3561	3647	3561
   q11	421	227	214	214
   q12	631	310	306	306
   q13	17798	2863	2840	2840
   q14	273	242	253	242
   q15	492	466	456	456
   q16	484	398	399	398
   q17	969	560	529	529
   q18	7330	6670	6549	6549
   q19	5606	1502	1396	1396
   q20	551	296	285	285
   q21	6195	3544	3580	3544
   q22	365	324	308	308
   Total cold run time: 112703 ms
   Total hot run time: 38466 ms
   
   ----- Round 2, with runtime_filter_mode=off -----
   q1	4166	4071	4068	4068
   q2	325	229	222	222
   q3	2997	2832	2812	2812
   q4	1851	1552	1542	1542
   q5	5208	5250	5237	5237
   q6	197	116	115	115
   q7	2259	1824	1808	1808
   q8	3156	3299	3307	3299
   q9	8595	8596	8541	8541
   q10	3747	3604	3650	3604
   q11	558	441	456	441
   q12	730	523	570	523
   q13	16904	2893	2833	2833
   q14	286	245	269	245
   q15	480	448	447	447
   q16	449	418	408	408
   q17	1751	1504	1467	1467
   q18	7507	7231	7204	7204
   q19	1843	1533	1517	1517
   q20	1880	1690	1719	1690
   q21	4771	4693	4729	4693
   q22	539	440	444	440
   Total cold run time: 70199 ms
   Total hot run time: 53156 ms
   ```
   </details>
   


-- 
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-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org


Re: [PR] [Fix](nereids) remove duplicate expr in grouping set [doris]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #32290:
URL: https://github.com/apache/doris/pull/32290#issuecomment-1999145872

   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 GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@doris.apache.org
For additional commands, e-mail: commits-help@doris.apache.org