apache / apache/iceberg-cpp

FileIO parameter should be arrow::fs::FileSystem ?

オープン
#307 コメント 14 件 リアクション 0 件 担当者 0 名 GitHub で見る
主要言語
C++
スター
221
フォーク
124
平均マージ
1日 16時間
マージ済み PR(30日)
21

説明

The ManifestWriter/ManifestListWriter API uses the` std::shared_ptr` parameter for the IO interface,
which actually should be an internal `arrow::ArrowFileSystemFileIO` (not publicly accessible).
It appears that only the `fs()` is actually utilized.
```c++
auto io = internal::checked_pointer_cast(options.io);
ICEBERG_ARROW_ASSIGN_OR_RETURN(auto output, io->fs()->OpenOutputStream(options.path));
```

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

このリポジトリのコントリビューションガイドは索引されていません

調査の方向性

ManifestWriter と ManifestListWriter の API から開始し、示されている checked_pointer_cast と fs()->OpenOutputStream の呼び出しを通じて options.io を追跡します。適切なファイルシステム指向のパラメーターを特定し、出力ストリームの作成が引き続き有効になるよう、関連する API の使用箇所を更新します。対象となる C++ テストまたはビルドが成功すれば完了です。

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

評価

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

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

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