Skip to content

[C++] Use JsonWriter consistently in LogicalType::ToJSON() #50830

Closed
Closed
[C++] Use JsonWriter consistently in LogicalType::ToJSON()#50830
Assignees
NathanChung4
Labels
Component: C++Type: enhancement
Milestone
26.0.0

Description

@Reranko05
Reranko05
opened on Aug 7, 2026

Describe the enhancement requested

LogicalType::ToJSON() currently mixes manual JSON string construction with ::arrow::json::JsonWriter.

Refactor it to use JsonWriter for building the entire JSON object to improve consistency and maintainability.

Parent Issue: #35460

Component(s)

C++

Activity

  1. Reranko05
    added
    Type: enhancement
    on Aug 7, 2026
  2. github-actions
    added
    Component: C++
    on Aug 7, 2026
  3. Reranko05
    added a parent issue on Aug 7, 2026

  4. NathanChung4 commented on Aug 11, 2026

    @NathanChung4
    NathanChung4
    on Aug 11, 2026
    Contributor

    take

  5. github-actions
    assigned
    NathanChung4
    on Aug 11, 2026
  6. NathanChung4
    added a commit that references this issue on Aug 15, 2026

    apacheGH-50830: [C++][Parquet] Use JsonWriter for LogicalType::ToJSON()

    ddcbaa7
  7. pitrou
    added a commit that references this issue on Aug 27, 2026

    GH-50830: [C++][Parquet] Use JsonWriter for LogicalType::ToJSON() (#5…

    c8240bd
  8. pitrou
    added this to the 26.0.0 milestone on Aug 27, 2026

  9. pitrou commented on Aug 27, 2026

    @pitrou
    pitrou
    on Aug 27, 2026
    Member

    Issue resolved by pull request 50877
    #50877

  10. pitrou
    closed this as completedon Aug 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Component: C++Type: enhancement

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions