zclllyybb commented on PR #39255: URL: https://github.com/apache/doris/pull/39255#issuecomment-2285212587
> > > 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的报错本来就是兜底的，且老优化器的问题主要修复方式是不走老优化器） > > master 上还有老优化器么我记得不是删了么已经 还有，不清楚哪个版本删，而且我记得目前还不能直接删来着，有些语句新优化器都没支持上。。 -- 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]
