apache / apache/arrow-java

[Java] Make JniWrapper native method be public

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

説明

The goal is to integrate Gandiva into apache Drill project. Now drill and arrow has some differences at the column in memory representation. Drill has a 2.0 plan to integrate arrow. Now I want to do some prior work to integrate arrow lib into drill project. The first thing is to make JniWrapper's package level methods be public ones. Maybe we can rename this classes as UnsafeJniWrapper like java's Unsafe class to allow others to invoke these methods directly.

So what's your opinion about this advice ? If ok , I will submit a corresponding PR.

**Reporter**: [Weijie Tong](https://issues.apache.org/jira/browse/ARROW-4819)

**Note**: *This issue was originally created as [ARROW-4819](https://issues.apache.org/jira/browse/ARROW-4819). Please see the [migration documentation](https://github.com/apache/arrow/issues/14542) for further details.*

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

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

調査の方向性

まず JniWrapper を見つけ、そのパッケージレベルのメソッドと現在の呼び出し元を確認します。対象範囲がそれらのメソッドを public にすることだけなのか、クラス名を UnsafeJniWrapper に変更することも含むのかを明確にします。合意した API が外部パッケージから利用でき、Gandiva および Drill との統合に引き続き適していれば完了です。

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

評価

技術スタック
java
領域
api
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
説明が足りない
初心者へのやさしさ
25/100

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

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