morrySnow commented on PR #10194: URL: https://github.com/apache/doris/pull/10194#issuecomment-1158900682
> @924060929 is canceling the operator in the plan. Your rule conflicts with his logic. Do you need to wait for his incorporation before incorporating your pr? merge operator with plan is a BIG change for Nereids. IMO, it is better to make Nereids could WORK on SSB benchmark first and then apply this change. Although this will be a bit more work, but there are two advantages. First, we could verify that merging operator and plan into one data structure is feasible easily. Second, After we have verified the Nereids using the SSB benchmark, we can focus on work unrelated to the main process that will not be affected by this change. -- 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-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
