Skip to content

GH-50659: [C++][Dev] Implement ListView support in gdb_arrow.py - #50661

Open
KHARSHAVARDHAN-eng wants to merge 1 commit into
apache:mainfrom
KHARSHAVARDHAN-eng:gh-50659-listview-gdb
Open

GH-50659: [C++][Dev] Implement ListView support in gdb_arrow.py#50661
KHARSHAVARDHAN-eng wants to merge 1 commit into
apache:mainfrom
KHARSHAVARDHAN-eng:gh-50659-listview-gdb

Conversation

@KHARSHAVARDHAN-eng

@KHARSHAVARDHAN-eng KHARSHAVARDHAN-eng commented Jul 27, 2026

Copy link
Copy Markdown

Why are these changes needed?

gdb_arrow.py currently does not support pretty-printing for ListView and LargeListView data types.

This PR adds debugger support for these types while reusing the existing pretty-printer infrastructure.

What changes are included?

  • Add LIST_VIEW and LARGE_LIST_VIEW type registration.
  • Reuse the existing pretty-printer implementation where appropriate.
  • Add debugger fixtures.
  • Add GDB test coverage for the new types.

Are these changes tested?

Yes.

  • Added GDB tests.
  • Ran the relevant debugger tests locally.
  • Verified Python syntax.
  • Verified pre-commit hooks.
  • Verified formatting.

Are there any user-facing changes?

No.

This change only improves the developer debugging experience.

@github-actions

Copy link
Copy Markdown

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

@Reranko05

Copy link
Copy Markdown
Contributor

Could you update the PR description to follow the standard PR template? It looks like the placeholder text is still present. Replacing it with the rationale, changes, testing, and removing the unused sections would make the PR easier to review.

Thanks for contributing!

@KHARSHAVARDHAN-eng

Copy link
Copy Markdown
Author

Thanks for pointing that out! I've updated the PR description to follow the standard template and removed the placeholder text.

@github-actions

Copy link
Copy Markdown

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

Comment thread cpp/gdb_arrow.py

@Reranko05 Reranko05 Jul 27, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This overlaps with GH-50662 since both PRs modify the same Type enum definition in gdb_arrow.py. They will likely conflict when merged. Could you rebase one after the other lands (or combine them if appropriate)?

@KHARSHAVARDHAN-eng

Copy link
Copy Markdown
Author

Thanks for pointing that out!

I'll keep these as separate PRs since they correspond to separate issues. Once one of them lands, I'll rebase the other on top of the updated main to resolve the overlap.

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.

2 participants