Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GH-40366: [C++] Remove const qualifier from Buffer::mutable_span_as #40367

Merged

Conversation

sjperkins
Copy link
Contributor

@sjperkins sjperkins commented Mar 5, 2024

Rationale for this change

The const qualifier on Buffer:mutable_span_as prevents it from calling the non-const Buffer::mutable_data_as

What changes are included in this PR?

See issue title

Are these changes tested?

No, I believe this is a simple oversight

Are there any user-facing changes?

Yes, Buffer::mutable_span_as loses it's const qualifier

Copy link

github-actions bot commented Mar 5, 2024

⚠️ GitHub issue #40366 has been automatically assigned in GitHub to PR creator.

@pitrou
Copy link
Member

pitrou commented Mar 5, 2024

@github-actions crossbow submit -g cpp

@pitrou pitrou requested a review from bkietz March 5, 2024 17:07
Copy link

github-actions bot commented Mar 5, 2024

Revision: 58b3bf6

Submitted crossbow builds: ursacomputing/crossbow @ actions-67a429d9a4

Task Status
test-alpine-linux-cpp GitHub Actions
test-build-cpp-fuzz GitHub Actions
test-conda-cpp GitHub Actions
test-conda-cpp-valgrind Azure
test-cuda-cpp GitHub Actions
test-debian-11-cpp-amd64 GitHub Actions
test-debian-11-cpp-i386 GitHub Actions
test-fedora-39-cpp GitHub Actions
test-ubuntu-20.04-cpp GitHub Actions
test-ubuntu-20.04-cpp-bundled GitHub Actions
test-ubuntu-20.04-cpp-minimal-with-formats GitHub Actions
test-ubuntu-20.04-cpp-thread-sanitizer GitHub Actions
test-ubuntu-22.04-cpp GitHub Actions
test-ubuntu-22.04-cpp-20 GitHub Actions
test-ubuntu-22.04-cpp-no-threading GitHub Actions
test-ubuntu-24.04-cpp GitHub Actions
test-ubuntu-24.04-cpp-gcc-14 GitHub Actions

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting review Awaiting review labels Mar 5, 2024
Copy link
Member

@kou kou left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@kou kou merged commit ee1a8c3 into apache:main Mar 6, 2024
35 of 37 checks passed
@kou kou removed the awaiting merge Awaiting merge label Mar 6, 2024
@github-actions github-actions bot added the awaiting merge Awaiting merge label Mar 6, 2024
Copy link

After merging your PR, Conbench analyzed the 0 benchmarking runs that have been run so far on merge-commit ee1a8c3.

None of the specified runs were found on the Conbench server.

The full Conbench report has more details.

@sjperkins sjperkins deleted the remove-const-qualifer-from-mutable_span_as branch March 6, 2024 08:51
mapleFU pushed a commit to mapleFU/arrow that referenced this pull request Mar 7, 2024
…n_as (apache#40367)

### Rationale for this change

The const qualifier on `Buffer:mutable_span_as` prevents it from calling the non-const `Buffer::mutable_data_as`

### What changes are included in this PR?

See issue title

### Are these changes tested?

No, I believe this is a simple oversight 

### Are there any user-facing changes?

Yes, `Buffer::mutable_span_as` loses it's const qualifier

* GitHub Issue: apache#40366

Authored-by: Simon Perkins <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
thisisnic pushed a commit to thisisnic/arrow that referenced this pull request Mar 8, 2024
…n_as (apache#40367)

### Rationale for this change

The const qualifier on `Buffer:mutable_span_as` prevents it from calling the non-const `Buffer::mutable_data_as`

### What changes are included in this PR?

See issue title

### Are these changes tested?

No, I believe this is a simple oversight 

### Are there any user-facing changes?

Yes, `Buffer::mutable_span_as` loses it's const qualifier

* GitHub Issue: apache#40366

Authored-by: Simon Perkins <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants