wForget commented on PR #3216: URL: https://github.com/apache/incubator-kyuubi/pull/3216#issuecomment-1211730776
> redirect unrecognized modes to NONE with a warning message, it has a deterministic behavior which does not rely on kyuubi.operation.plan.only.excludes When we set a wrong plan mode he won't have any exception, then we execute the insert command successfully, which seems dangerous. > this test works because of kyuubi.operation.plan.only.excludes. > So this is actually a side-effect and indeterministic. It looks like this, but if we need to switch plan mode normally, we also rely on `kyuubi.operation.plan.only.excludes`: ``` set kyuubi.operation.plan.only.mode=parse; set kyuubi.operation.plan.only.mode=none; ``` -- 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]
