Skip to content

[C++] Incorrect Logic for AdaptiveUIntBuilder::AppendValues #51144

Closed

Description

@andishgar

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

The following code

TEST(TestAdaptiveUIntBuilderWithStartIntSize, MyTest) {
  AdaptiveUIntBuilder builder;
  ASSERT_OK(builder.Append(1));
  ASSERT_OK(builder.Append(2));
  ASSERT_OK(builder.Append(3));
  ASSERT_OK(builder.Append(4));
  std::vector<uint64_t> values{5,6,7,8};
  ASSERT_OK(builder.AppendValues(values.data(),4));
  auto array = builder.Finish().ValueOrDie();
  ARROW_LOGGER_INFO("",array->ToString());
}

produces the following result because the pending data is not committed here:

[
  1,
  2,
  3,
  4
]

Component(s)

C++

Activity

  1. added a commit that references this issue on Sep 3, 2026
  2. added this to the 26.0.0 milestone on Sep 3, 2026
  3. pitrou commented on Sep 3, 2026

    @pitrou
    Member

    Issue resolved by pull request 51146
    #51146

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions