Skip to content

GH-51395: [C++][Docs] Replace RapidJSON with simdjson in row converter example - #51396

Draft
Reranko05 wants to merge 3 commits into
apache:mainfrom
Reranko05:example-doc
Draft

Reranko05 wants to merge 3 commits into
apache:mainfrom
Reranko05:example-doc

Conversation

@Reranko05

@Reranko05 Reranko05 commented Sep 19, 2026

Copy link
Copy Markdown
Collaborator

Rationale for this change

This PR continues the simdjson migration by replacing RapidJSON usage in the C++ row converter example with simdjson and Arrow's existing JsonWriter.

Changes

  • Replace RapidJSON DOM parsing with simdjson's DOM API.
  • Replace RapidJSON JSON generation with Arrow's JsonWriter.
  • Preserve the existing row-to-Arrow and Arrow-to-row conversion flow.
  • Update the example to link against arrow::simdjson.

Was AI used for this PR?

In accordance to the AI generation guidelines, please disclose below whether and how AI was used in this PR.

PR code and description written by:

  • Human
  • AI

Reviewed before submission by:

  • Human
  • AI
  • Not reviewed

@Reranko05 Reranko05 added the CI: Extra: C++ Run extra C++ CI label Sep 19, 2026
@Reranko05 Reranko05 changed the title GH-51395: [C++] Replace RapidJSON with Simdjson in row converter example GH-51395: [C++] Replace RapidJSON with simdjson in row converter example Sep 19, 2026
@Reranko05 Reranko05 changed the title GH-51395: [C++] Replace RapidJSON with simdjson in row converter example GH-51395: [C++][Docs] Replace RapidJSON with simdjson in row converter example Sep 19, 2026
@Reranko05

Copy link
Copy Markdown
Collaborator Author

Will mark this PR Ready to Review after #51038 is merged.

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.

1 participant