apache / apache/arrow-java

Offer an un-shaded version of the Flight SQL JDBC driver

オープン
#161 コメント 3 件 リアクション 1 件 担当者 0 名 GitHub で見る
主要言語
Java
スター
94
フォーク
152
平均マージ
3日 16時間
マージ済み PR(30日)
11

説明

### Describe the enhancement requested

The Arrow Flight SQL JDBC driver shades and relocates all of its Maven dependencies into the driver JAR. While this is probably good and desired for folks who need to download the JAR and install it into whatever tool they're using to connect to something, it can cause an issue for those who are embedding the driver within their own service.

As an example, we're using [OpenTelemetry's Java auto instrumentation agent](https://opentelemetry.io/docs/instrumentation/java/automatic/). This agent modifies bytecode as it's loaded to include OpenTelemetry instrumentation for distributed tracing. It has built in support for gRPC clients and servers. Unfortunately, because the Arrow Flight SQL JDBC driver shades and relocates the gRPC libraries, they no longer utilize the same package name as what the Java agent expects. Subsequently, OpenTelemetry does not instrument the client calls and can not perform context propagation from the client to the server.

Would it be possible to publish both a shaded+relocated version of the driver and an un-shaded version of the driver?

The Maven Shade plugin supports building and publishing both artifacts via the [shadedArtifactAttached](https://maven.apache.org/plugins/maven-shade-plugin/examples/attached-artifact.html) configuration option. The JDBC driver has this option explicitly disabled for some reason.

### Component(s)

Java

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

Flight SQL JDBCドライバーのMaven Shade設定を調査し、まず shadedArtifactAttached を無効にしている設定を確認します。現在JDBCドライバーのアーティファクトがどのようにビルドおよび公開されているかを確認し、そのうえで、既存のドライバーパッケージングを壊すことなく、shaded/relocated アーティファクトとunshadedアーティファクトの両方が生成されることを検証します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
grpc, java
領域
build-system
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。