[JAVA] flight-sql-jdbc-driver V16.0.0 shading relocates slf4j dependency
- Ngôn ngữ chính
- Java
- Star
- 94
- Fork
- 152
- Merge trung bình
- 3 ngày 16 giờ
- Pull request đã merge (30 ngày)
- 11
Mô tả
### 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
Hướng dẫn đóng góp
Hướng nghiên cứu
Bắt đầu trong java/flight/flight-sql-jdbc-driver/pom.xml, tại khu vực các shading relocation được liên kết trong issue, và so sánh cấu hình 16.0.0 với phiên bản 15.x.x. Kiểm tra cách các pattern com. và org. ảnh hưởng đến SLF4J và các import @Slf4j. Hoàn tất khi driver không còn relocation ngoài dự kiến dependency logging hướng tới người dùng, đồng thời vẫn giữ shading như dự định.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- java
- Lĩnh vực
- build-system
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 35/100