search

helidon-io / helidon

Java libraries for writing microservices
https://helidon.io
Apache License 2.0
3.44k stars 562 forks source link

4.x: Use System.Logger instead of JUL where applicable #7792 #8791

Closed jbescos closed 1 month ago

jbescos commented 1 month ago

Description

https://github.com/helidon-io/helidon/issues/7792

Documentation

N/A

jbescos commented 1 month ago

There are some leftovers where I was not able to remove because of any of the next reasons:

  1. There is a strong dependency to JUL in one public method and I cannot remove it to stay backwards compatible. https://github.com/helidon-io/helidon/issues/7792#issuecomment-2124542588
  2. Class requires dependencies on JUL (handlers, log records, etc)
$ grep -R 'requires\ java\.logging' --include 'module-info.java' ./
./webserver/observe/log/src/main/java/module-info.java:    requires java.logging;
./webserver/access-log/src/main/java/module-info.java:    requires java.logging;
./logging/jul/src/main/java/module-info.java:    requires java.logging;
./helidon/src/main/java/module-info.java:    requires java.logging;
./messaging/messaging/src/main/java/module-info.java:    requires java.logging;
./integrations/cdi/jta-weld/src/main/java/module-info.java:    requires java.logging;
./integrations/micrometer/micrometer/src/main/java/module-info.java:    requires java.logging;
./tracing/provider-tests/src/main/java/module-info.java:    requires java.logging;
./microprofile/openapi/src/main/java/module-info.java:    requires java.logging; // logging required for SnakeYAML logging workaround
./common/reactive/src/main/java/module-info.java:    requires java.logging;
./common/configurable/src/main/java/module-info.java:    requires java.logging;
./common/testing/junit5/src/main/java/module-info.java:    requires java.logging;