Skip to content

GH-51140: [C++] Require simdjson 4.3.0 for fractured_json APIs - #51142

Merged
kou merged 1 commit into
apache:mainfrom
tadeja:simdjson_required_v4.3.0
Sep 3, 2026
Merged

GH-51140: [C++] Require simdjson 4.3.0 for fractured_json APIs#51142
kou merged 1 commit into
apache:mainfrom
tadeja:simdjson_required_v4.3.0

Conversation

@tadeja

@tadejatadeja commented Sep 2, 2026

Copy link
Copy Markdown
Member

Rationale for this change

Fix#51140, CMake configuration should not accept simdjson earlier than v4.3.0.

What changes are included in this PR?

Set ARROW_SIMDJSON_REQUIRED_VERSION to "4.3.0"

Are these changes tested?

Yes. Locally, CMake configuration rejects system simdjson 4.2.3 now
CMake Error at cmake_modules/ThirdpartyToolchain.cmake:316 (message):
Couldn't find simdjson >= 4.3.0

Are there any user-facing changes?

No.

@github-actionsgithub-actionsBot added the awaiting committer review Awaiting committer review label Sep 2, 2026
@github-actions

Copy link
Copy Markdown

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

@tadeja
tadeja marked this pull request as ready for review September 2, 2026 15:09
CopilotAI lite review requested due to automatic review settings September 2, 2026 15:09
@tadeja

Copy link
Copy Markdown
MemberAuthor

@github-actions crossbow submit test-build-vcpkg-win test-ubuntu-22.04-cpp-bundled

CopilotAI left a comment

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.

🟢 Approval recommended

The change is narrowly scoped, matches the documented minimum simdjson API requirement, and reduces the risk of configuration-time acceptance of incompatible system versions.

Pull request overview

Updates the C++ third-party dependency configuration to require simdjson >= 4.3.0, matching the minimum version needed for the fractured_json APIs used in Arrow and preventing misconfigured builds with older system simdjson installs.

Changes:

  • Bump ARROW_SIMDJSON_REQUIRED_VERSION from 4.0.0 to 4.3.0 in the CMake toolchain dependency resolution.
File summaries
FileDescription
cpp/cmake_modules/ThirdpartyToolchain.cmakeRaises the minimum accepted simdjson version to avoid build failures when system simdjson is < 4.3.0.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@github-actions

Copy link
Copy Markdown

Revision: f3018b3

Submitted crossbow builds: ursacomputing/crossbow @ actions-0a31401513

TaskStatus
test-build-vcpkg-winGitHub Actions
test-ubuntu-22.04-cpp-bundledGitHub Actions

@tadeja
tadeja requested a review from kouSeptember 2, 2026 17:51
kou
kou approved these changes Sep 3, 2026

@koukou left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

+1

@github-actionsgithub-actionsBot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Sep 3, 2026
@kou
kou merged commit 8ebb636 into apache:mainSep 3, 2026
55 of 57 checks passed
@koukou removed the awaiting merge Awaiting merge label Sep 3, 2026
@rok

rok commented Sep 3, 2026

Copy link
Copy Markdown
Member

Regarding simdjson version - we had a version discussion here already - #50568 (comment). TLDR: ubuntu lags, only 26.0.10 has 4.6.8, while 26.0.4 has 4.2.4.

Sign up for freeto 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.

[C++] Require simdjson 4.3.0 for fractured_json APIs

4 participants

@tadeja@rok@kou