FileIO parameter should be arrow::fs::FileSystem ?
未关闭
- 主要语言
- C++
- 星标
- 221
- 派生
- 124
- 平均合并
- 1 天 16 小时
- 30 天内合并 PR
- 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