[JAVA] flight-sql-jdbc-driver V16.0.0 shading relocates slf4j dependency
- Vorherrschende Sprache
- Java
- Sterne
- 94
- Forks
- 152
- Ø Merge
- 3 T. 16 Std.
- Gemergte PRs (30 T.)
- 11
Beschreibung
### Describe the bug, including details regarding any error messages, version, and platform.
When I upgrade to version 16.0.0, suddenly my @Slf4j annotation did no longer work. When I checked for manually creating a LoggerFactory, I noticed that the only imports Intellj could find were unter the path of:
import org.apache.arrow.driver.jdbc.shaded.*;
I then checked the pom.xml of the flight-sql-jdbc-driver and noticed the shading relocations you configured:
https://github.com/apache/arrow/blob/7cd9c6fbd313c9afa0673d85415a84fd199951c9/java/flight/flight-sql-jdbc-driver/pom.xml#L98
```
com.
org.apache.arrow.driver.jdbc.shaded.com.
com.sun.**
org.
org.apache.arrow.driver.jdbc.shaded.org.
org.apache.arrow.driver.jdbc.**
org.apache.arrow.flight.name
org.apache.arrow.flight.version
org.apache.arrow.flight.jdbc-driver.name
org.apache.arrow.flight.jdbc-driver.version
```
I am not an expert on the shading plugin of maven, but it seems to me it moves around all dependencies below the com. and org. pattern. Even if I exclude those dependencies from the flight-sql-jdbc-driver, it does not help.
If I go down to version 15.x.x it works again.
### Component(s)
Java
Beitragsleitfaden
Rechercherichtung
Beginne in java/flight/flight-sql-jdbc-driver/pom.xml bei den im Issue verknüpften Shading-Relocations und vergleiche die Konfiguration von 16.0.0 mit der Version 15.x.x. Prüfe, wie sich die Muster com. und org. auf SLF4J und die @Slf4j-Imports auswirken. Fertig ist die Änderung, wenn der Treiber die benutzerseitige Logging-Abhängigkeit nicht mehr unerwartet relocatet und sein vorgesehenes Shading beibehält.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- java
- Bereich
- build-system
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100