thswlsqls opened a new pull request, #8681:
URL: https://github.com/apache/paimon/pull/8681
### Purpose
- `KvQueryServer.shutdown()` logged failures as `LOG.warn("{} shutdown
failed: {}", getServerName(), e)`. The second `{}` consumes `e`, so SLF4J
renders it via `toString()` and drops the stack trace, making shutdown failures
hard to diagnose.
- Dropping that placeholder passes `e` as the trailing throwable, so SLF4J
logs the full stack trace.
- Matches the sibling `KvQueryClient.shutdown()`, which already logs the
throwable with its stack trace.
### Tests
- Logging-only change, no behavior change — no test added. `mvn -pl
paimon-service/paimon-service-runtime clean install` (JDK 11) passed checkstyle
+ spotless + rat with 12 unit tests green (KvQueryTableTest, NetworkClientTest,
NetworkServerTest, ClientHandlerTest).
--
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]
