[JAVA] flight-sql-jdbc-driver V16.0.0 shading relocates slf4j dependency
- Lenguaje dominante
- Java
- Estrellas
- 94
- Forks
- 152
- Merge medio
- 3 d 16 h
- PR fusionados (30 d)
- 11
Descripción
### 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
Guía de contribución
Línea de trabajo
Comienza en java/flight/flight-sql-jdbc-driver/pom.xml, alrededor de las relocations de shading enlazadas en el issue, y compara la configuración de 16.0.0 con la versión 15.x.x. Comprueba cómo afectan los patrones com. y org. a SLF4J y a los imports de @Slf4j. Se considera terminado cuando el driver ya no reubica inesperadamente la dependencia de logging orientada al usuario, manteniendo al mismo tiempo su shading previsto.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- java
- Área
- build-system
- Tipo de issue
- Error
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100