maple525866 commented on PR #7456: URL: https://github.com/apache/incubator-seata/pull/7456#issuecomment-2991470274
> > 我们需要解决的主要问题是测试类中的 REPLACE 语句不受支持，并抛出 ParserException，从而导致 Maven 构建错误。 > > 我的解决方案：修改DruidSQLRecognizerFactoryImpl.java，将Druid的ParserException转换为Seata的NotSupportYetException。 > > 同时，为了兼顾之前的测试类，使用反射来检查异常类型，避免直接引用ParserException > > 结果：GitHub Action 构建成功；统一异常处理 - 所有不支持的 SQL 语法都会抛出 NotSupportYetException；一致的测试逻辑 - 所有数据库类型都使用相同的测试断言；保持兼容性 - 其他测试（如 DruidIsolationTest）正常运行 > > 我没有仔细看测试代码😅，将其更改为`NotSupportYetException`确实看起来有效！ > > 我认为当前的方法也很好，但是如何仅将抛出 ParserException 的方法包装在 try-catch 块中？ > > * 目前，整个方法都包裹在 try-catch 块中，我认为这会影响可读性。 done -- 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: notifications-unsubscr...@seata.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@seata.apache.org For additional commands, e-mail: notifications-h...@seata.apache.org
