apache / apache/arrow-java

[Java] JDBC to Arrow converter iterator should reuse the same VectorSchemaRoot

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

説明

When originally reviewing the code I suggested a new VectorSchemaRoot on each call to the iterator.  After further discussions on the mailing list, it seems that this is an anit-pattern for working with VectorSchemaRoot, we should update the code to update a single VectorSchemaRoot.

 

After this change it should be easier to use JDBC converter with other components of the library (i.e. filewriter) which also make use of a single VectorSchemaRoot.

**Reporter**: [Micah Kornfield](https://issues.apache.org/jira/browse/ARROW-9614) / @emkornfield

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

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

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

調査の方向性

まず Java JDBC converter の iterator を見つけ、VectorSchemaRoot の扱いを issue で言及されている filewriter コンポーネントと比較します。iterator が現在どのように roots を作成しているかを確認し、関連するテストを特定します。converter が 1 つの VectorSchemaRoot を再利用し、単一の root を想定するコンポーネントで動作すれば完了です。

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

評価

技術スタック
java
領域
databases
issue の種類
リファクタリング
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
38/100

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

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