Skip to content

Conversation

@pitrou
Copy link
Member

@pitrou pitrou commented Mar 31, 2025

What changes are included in this PR?

Update bundled mimalloc version from 2.0.6 to 2.2.4. This will also help fix the CMake 4.0 compatibility issues.

Are these changes tested?

Yes, by CI builds.

Are there any user-facing changes?

No.

@github-actions
Copy link

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

@pitrou
Copy link
Member Author

pitrou commented Mar 31, 2025

@github-actions crossbow submit -g cpp -g python

@github-actions

This comment was marked as outdated.

@pitrou pitrou force-pushed the gh45978-bump-mimalloc branch from c9a3cbe to 161d91b Compare March 31, 2025 14:12
@pitrou
Copy link
Member Author

pitrou commented Mar 31, 2025

@github-actions crossbow submit -g cpp -g python

@github-actions

This comment was marked as outdated.

@pitrou
Copy link
Member Author

pitrou commented Mar 31, 2025

I've opened microsoft/mimalloc#1056 for the Alpine Linux failure but I don't think it should be holding this.

@pitrou pitrou force-pushed the gh45978-bump-mimalloc branch 2 times, most recently from 2185744 to 222ec0b Compare March 31, 2025 15:05
Copy link
Member

@assignUser assignUser left a comment

Choose a reason for hiding this comment

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

It looks like we also need to forward snappy
Edit: #45980

Comment on lines 299 to 311
ARROW_SUBSTRAIT: ON
ARROW_USE_GLOG: OFF
ARROW_VERBOSE_THIRDPARTY_BUILD: OFF
ARROW_VERBOSE_THIRDPARTY_BUILD: ON
Copy link
Member

Choose a reason for hiding this comment

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

I assume you'll revert these before merge?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, this was to debug the build/link errors on Windows which are now fixed.

@github-actions github-actions bot added awaiting committer review Awaiting committer review and removed awaiting review Awaiting review labels Mar 31, 2025
@pitrou pitrou force-pushed the gh45978-bump-mimalloc branch from 8241964 to 97490ec Compare June 11, 2025 19:14
@pitrou
Copy link
Member Author

pitrou commented Jun 11, 2025

@github-actions crossbow submit -g cpp -g python

@github-actions
Copy link

Revision: 97490ec58b0ed1c9993b607aa0046bc7fbfde66a

Submitted crossbow builds: ursacomputing/crossbow @ actions-21dfeaf22e

Task Status
example-cpp-minimal-build-static GitHub Actions
example-cpp-minimal-build-static-system-dependency GitHub Actions
example-cpp-tutorial GitHub Actions
example-python-minimal-build-fedora-conda GitHub Actions
example-python-minimal-build-ubuntu-venv GitHub Actions
test-build-cpp-fuzz GitHub Actions
test-conda-cpp GitHub Actions
test-conda-cpp-valgrind GitHub Actions
test-conda-python-3.10 GitHub Actions
test-conda-python-3.10-hdfs-2.9.2 GitHub Actions
test-conda-python-3.10-hdfs-3.2.1 GitHub Actions
test-conda-python-3.10-pandas-latest-numpy-latest GitHub Actions
test-conda-python-3.11 GitHub Actions
test-conda-python-3.11-dask-latest GitHub Actions
test-conda-python-3.11-dask-upstream_devel GitHub Actions
test-conda-python-3.11-hypothesis GitHub Actions
test-conda-python-3.11-pandas-latest-numpy-1.26 GitHub Actions
test-conda-python-3.11-pandas-latest-numpy-latest GitHub Actions
test-conda-python-3.11-pandas-nightly-numpy-nightly GitHub Actions
test-conda-python-3.11-pandas-upstream_devel-numpy-nightly GitHub Actions
test-conda-python-3.11-spark-master GitHub Actions
test-conda-python-3.12 GitHub Actions
test-conda-python-3.12-cpython-debug GitHub Actions
test-conda-python-3.13 GitHub Actions
test-conda-python-3.9 GitHub Actions
test-conda-python-3.9-pandas-1.1.3-numpy-1.19.5 GitHub Actions
test-conda-python-emscripten GitHub Actions
test-cuda-cpp-ubuntu-22.04-cuda-11.7.1 GitHub Actions
test-cuda-python-ubuntu-22.04-cuda-11.7.1 GitHub Actions
test-debian-12-cpp-amd64 GitHub Actions
test-debian-12-cpp-i386 GitHub Actions
test-debian-12-python-3-amd64 GitHub Actions
test-debian-12-python-3-i386 GitHub Actions
test-fedora-39-cpp GitHub Actions
test-fedora-39-python-3 GitHub Actions
test-ubuntu-22.04-cpp GitHub Actions
test-ubuntu-22.04-cpp-20 GitHub Actions
test-ubuntu-22.04-cpp-bundled GitHub Actions
test-ubuntu-22.04-cpp-emscripten GitHub Actions
test-ubuntu-22.04-cpp-no-threading GitHub Actions
test-ubuntu-22.04-python-3 GitHub Actions
test-ubuntu-22.04-python-313-freethreading GitHub Actions
test-ubuntu-24.04-cpp GitHub Actions
test-ubuntu-24.04-cpp-bundled-offline GitHub Actions
test-ubuntu-24.04-cpp-gcc-13-bundled GitHub Actions
test-ubuntu-24.04-cpp-gcc-14 GitHub Actions
test-ubuntu-24.04-cpp-minimal-with-formats GitHub Actions
test-ubuntu-24.04-cpp-thread-sanitizer GitHub Actions
test-ubuntu-24.04-python-3 GitHub Actions

@pitrou pitrou added the CI: Extra Run extra CI label Jun 11, 2025
@pitrou
Copy link
Member Author

pitrou commented Jun 11, 2025

@github-actions crossbow submit wheelcp313*

Update bundled mimalloc version from 2.0.6 to 2.2.4.
This will also help fix the CMake 4.0 compatibility issues.
@pitrou pitrou force-pushed the gh45978-bump-mimalloc branch from 97490ec to 1fe3480 Compare June 11, 2025 20:03
@github-actions
Copy link

Revision: 1fe3480

Submitted crossbow builds: ursacomputing/crossbow @ actions-dc8e7b294f

Task Status
wheel-macos-monterey-cp313-cp313-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313-arm64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-amd64 GitHub Actions
wheel-macos-monterey-cp313-cp313t-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313-arm64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-amd64 GitHub Actions
wheel-manylinux-2-28-cp313-cp313t-arm64 GitHub Actions
wheel-manylinux-2014-cp313-cp313-amd64 GitHub Actions
wheel-manylinux-2014-cp313-cp313-arm64 GitHub Actions
wheel-manylinux-2014-cp313-cp313t-amd64 GitHub Actions
wheel-manylinux-2014-cp313-cp313t-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313-arm64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-amd64 GitHub Actions
wheel-musllinux-1-2-cp313-cp313t-arm64 GitHub Actions
wheel-windows-cp313-cp313-amd64 GitHub Actions
wheel-windows-cp313-cp313t-amd64 GitHub Actions

@pitrou
Copy link
Member Author

pitrou commented Jun 11, 2025

This is ready for review @kou . CI looks good except for a probably unrelated failure in the Python emscripten build.

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

@github-actions github-actions bot added awaiting merge Awaiting merge and removed awaiting committer review Awaiting committer review labels Jun 11, 2025
@pitrou pitrou force-pushed the gh45978-bump-mimalloc branch from 5af7504 to cc5d86e Compare June 12, 2025 05:29
@github-actions github-actions bot added awaiting changes Awaiting changes and removed awaiting merge Awaiting merge labels Jun 12, 2025
@pitrou pitrou merged commit 0826be5 into apache:main Jun 12, 2025
34 of 35 checks passed
@pitrou pitrou removed the awaiting changes Awaiting changes label Jun 12, 2025
@pitrou pitrou deleted the gh45978-bump-mimalloc branch June 12, 2025 08:00
@conbench-apache-arrow
Copy link

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

There were 76 benchmark results with an error:

There were no benchmark performance regressions. 🎉

The full Conbench report has more details. It also includes information about 32 possible false positives for unstable benchmarks that are known to sometimes produce them.

alinaliBQ pushed a commit to Bit-Quill/arrow that referenced this pull request Jun 17, 2025
### What changes are included in this PR?

Update bundled mimalloc version from 2.0.6 to 2.2.4. This will also help fix the CMake 4.0 compatibility issues.

### Are these changes tested?

Yes, by CI builds.

### Are there any user-facing changes?

No.

* GitHub Issue: apache#45978

Authored-by: Antoine Pitrou <antoine@python.org>
Signed-off-by: Antoine Pitrou <antoine@python.org>
@kou
Copy link
Member

kou commented Jul 4, 2025

@github-actions crossbow submit test-r-linux-sanitizers

@github-actions
Copy link

github-actions bot commented Jul 4, 2025

Revision: cc5d86e

Submitted crossbow builds: ursacomputing/crossbow @ actions-d6f9a3a132

Task Status
test-r-linux-sanitizers GitHub Actions

@Tom-Newton
Copy link
Contributor

I think this change has caused a memory leak when reading parquet #47266

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants