zclllyybb commented on code in PR #56304:
URL: https://github.com/apache/doris/pull/56304#discussion_r2374337296
##########
regression-test/suites/partition_p0/auto_partition/test_auto_partition_behavior.groovy:
##########
@@ -225,12 +225,11 @@ suite("test_auto_partition_behavior") {
"replication_num" = "1"
);
"""
- test{
- sql """insert into `long_value` values
("jwklefjklwehrnkjlwbfjkwhefkjhwjkefhkjwehfkjwehfkjwehfkjbvkwebconqkcqnocdmowqmosqmojwnqknrviuwbnclkmwkj");"""
- def exception_str = isGroupCommitMode() ? "s length is over limit of
50." : "Partition name's length is over limit of 50."
- exception exception_str
- }
+ sql """insert into `long_value` values
("jwklefjklwehrnkjlwbfjkwhefkjhwjkefhkjwehfkjwehfkjwehfkjbvkwebconqkcqnocdmowqmosqmojwnqknrviuwbnclkmwkj");"""
Review Comment:
我们直接加一组测试吧，端到端的建表+插入，包含中文字符，assert一下partition名字和auto_partition_name算出来的结果一致
--
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]
