sunnysabor commented on PR #7408: URL: https://github.com/apache/shenyu/pull/7408#issuecomment-5939355524
The three PostgreSQL `TagMapperTest` errors share a production root cause. `PostgreSqlUpdateInterceptor` intercepts every executor update and dereferences `parameter.getClass()` unconditionally; valid no-parameter statements such as `TagMapper.deleteAllData()` pass null and fail before SQL execution. I added a null guard while preserving the existing `dateUpdated` mutation for parameterized updates, plus an interceptor test verifying that the null parameter is forwarded to the executor. `TagMapperTest` (10 cases) and `PostgreSqlUpdateInterceptorTest` (4 cases) now pass together against PostgreSQL 14; Checkstyle passes. Commit `93dc3e983` is pushed to this Draft PR. -- 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]
