zclllyybb commented on PR #39255: URL: https://github.com/apache/doris/pull/39255#issuecomment-2284089128
> The test only guarantee exception is thrown from FE. Modification on BE is not covered, and this is important, since we need to upgrade cluster. > > When the pr is picked to 2.1, should add more tests to cover non-nereids planner. 实际上，最后面加的那个testcase，就是接收的从BE报的错： ```sql mysql [test]>set enable_nereids_planner=false; Query OK, 0 rows affected (0.02 sec) mysql [test]>select random(1,array_size(split_by_string(fcst_emp,','))) from test_random; ERROR 1105 (HY000): errCode = 2, detailMessage = (10.16.10.8)[INVALID_ARGUMENT]The param of rand function must be literal ``` 剩下的交给升降级测试，回归里只能测到这些了（BE的报错本来就是兜底的，且老优化器的问题主要修复方式是不走老优化器） -- 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]
