diqiu50 opened a new pull request, #12635: URL: https://github.com/apache/gravitino/pull/12635
### What changes were proposed in this pull request? Switch trino-connector's logging from SLF4J/Log4j2 to `io.airlift.log.Logger`, matching Trino's own connectors. ### Why are the changes needed? Trino loads the connector plugin in an isolated classloader. Log4j2 is classloader-scoped and the plugin jar ships no config, so it falls back to Log4j2's default (root ERROR, console) — most log output is silently dropped instead of reaching `var/log/server.log`. `io.airlift.log.Logger` wraps `java.util.logging`, a JVM-wide singleton unaffected by classloader isolation. Fix: #12634 ### Does this PR introduce any user-facing change? No. Trino admins will now see Gravitino connector logs in `var/log/server.log`. ### How was this patch tested? - Compiled all trino-connector modules (base + 5 version-range variants) - `./gradlew :trino-connector:trino-connector:test` - `./build.sh sp` -- 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]
