apache / apache/iceberg-cpp

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

未关闭
#307 14 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
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

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。