FileIO parameter should be arrow::fs::FileSystem ?
オープン
- 主要言語
- 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