morrySnow commented on code in PR #64793:
URL: https://github.com/apache/doris/pull/64793#discussion_r3479089608
##########
regression-test/suites/nereids_p0/local_shuffle/test_local_shuffle_rqg_bugs.groovy:
##########
@@ -522,7 +522,7 @@ suite("test_local_shuffle_rqg_bugs") {
// then NLJ (LEFT JOIN table1 table3 ON pk > col_bigint_undef_signed)
// ============================================================
- logger.info("=== Bug 10: GLOBAL_HASH_SHUFFLE Rows mismatched - self-join +
NLJ (build 184181 case 906784672) ===")
+ logger.info("=== Bug 10: GLOBAL_HASH_SHUFFLE Rows mismatched - self-join +
NLJ (RQG testing case 906784672) ===")
def bug10_fe = sql """
SELECT /*+SET_VAR(use_serial_exchange=true,
parallel_pipeline_task_num=4,
Review Comment:
🟡 **New test section is missing a bug number/label**: All other test
sections in this file follow the convention `// Bug N: ...` with a specific
bug number. This new test (count(distinct)+std under serial exchange, inserted
between Bug 20 and Bug 21) lacks a bug number, making it inconsistent with the
rest of the file. Consider giving it a label (e.g., "Bug 20b" or a new bug
reference) for consistent organization.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
