Zhengcy05 commented on issue #38709: URL: https://github.com/apache/shardingsphere/issues/38709#issuecomment-4629279008
Hi👋, based on the discussion above, I plan to work on this issue with the following scope: - preserve or explicitly expose the `TEMPORARY` semantic from the MySQL parser / visitor to the DDL statement model; - add pre-execution validation for MySQL temporary table DDL; - return a clear unsupported-operation error for `CREATE TEMPORARY TABLE` and `DROP TEMPORARY TABLE` before the SQL is actually sent to the backend; - ensure temporary table DDL does not enter the current normal push-down metadata refresh flow; - add regression tests covering `CREATE TEMPORARY TABLE` / `DROP TEMPORARY TABLE`, to make sure we no longer hit an NPE and no longer end up in the current situation where the SQL has already been executed on the backend but fails later during metadata refresh. If this scope looks good, I’d like to take this issue and submit a PR following this 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]
