apache / apache/arrow-java

[Java] ArrowRecordBatch Body Length Computation Only Accounts for Last Buffer

未关闭
#188 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
Type: bug
主要语言
Java
星标
94
派生
152
平均合并
3 天 16 小时
30 天内合并 PR
11

描述

### Describe the bug, including details regarding any error messages, version, and platform.

Hi Arrow Community,

I believe `ArrowRecordBatch::computeBodyLength` has an implementation bug. It does the following

```
for (int i = 0; i < buffers.size(); i++) {
ArrowBuf buffer = buffers.get(i);
ArrowBuffer layout = buffersLayout.get(i);
size = layout.getOffset() + buffer.readableBytes();

// round up size to the next multiple of 8
size = DataSizeRoundingUtil.roundUpTo8Multiple(size);
}
return size;
```

Since `size` is being assigned and not incremented at first - does this mean computeBodyLength only accounts for the last buffer ?

### Component(s)

Java

贡献指南

打开贡献指南

调研方向

从 ArrowRecordBatch::computeBodyLength 开始,检查缓冲区布局和可读字节数如何影响返回值。使用多个缓冲区复现该计算,然后检查周围的 Java 代码或测试,以确认预期的正文长度语义。完成的标准是:计算涵盖每个缓冲区,并保留所需的 8 字节对齐。

由索引模型根据 Issue 内容生成。

评估

技术栈
java
领域
data-engineering
Issue 类型
缺陷
难度
2/5
预计耗时
1-3 小时
活跃度
停滞
描述清晰度
基本清楚
新手友好度
35/100

把新 issue 发到你的邮箱

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