zhangwenchao-123 commented on PR #1719: URL: https://github.com/apache/cloudberry/pull/1719#issuecomment-4411934827
LGTM to fallback temporary, but maybe the best way to fix the root cause is fixing the Sequence::pdsRequired, in my opinion, this path is a illegal path which should be pruned by pdsRequired restrition. What's more, maybe we can add restriction between CCTEReq and CCTEMap by checking the motion worker number, only when they have the same motion workers, CCTEMap::FSatisfies will return true and we will get a legal path. Prefer to the first path, it's more formal than second which looks like a remedial plan. -- 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]
