arrow: fix recipe when arrow/*:parquet=True

[robomics]

Specify library name and version: arrow/all

When the arrow package is built with arrow/*:parquet=True, arrow/*:with_thrift=True is also required.
Thrift is found by Arrow by calling FindThriftAlt.cmake, which uses some custom logic to define thrift::thrift when the project is compiled on Windows.

This causes the transitive_header=True specified in Thrift's recipe to be ignored.

conan-center-index/recipes/thrift/all/conanfile.py

Lines 69 to 70 in b4b8a8d

	def requirements(self):
	self.requires("boost/1.84.0", transitive_headers=True)

This PR patches FindThriftAlt.cmake so that find_package(Thrift) is always called.

Fixes #20082 (note that without the patch I am unable to build the package using MSVC regardless of the build_type).

While debugging the above issue I also realized that ARROW_WITH_RE2 was not explicitly set by the recipe, which on my machine causes compilation to fail for v11.0.0. This PR also addresses this issue.

Finally, I am bumping thrift, xsimd, and zstdto avoid version conflicts with other packages on cci.

---

• I've read the contributing guidelines.
• I've used a recent Conan client version close to the currently deployed.
• I've tried at least one configuration locally with the conan-center hook activated.

[valgur]

Thanks! Please also set parquet=True in default_options as well. It was only left disabled due to build issues in the last PR and this one should probably fix these.

[robomics]

Another thing I noticed, is that arrow always uses a vendored version of mimalloc: https://github.com/apache/arrow/blob/b2e8c33c86c819b167a1cbca834da3c9047a9350/cpp/cmake_modules/ThirdpartyToolchain.cmake#L2197-L2204

Should I drop mimalloc from the requirement list (but leave the option to toggle mimalloc on or off)?

[robomics]

@RubenRBS I think this is ready for review :).

The conan v2 pipeline failure seems to be due to the same issue described in #24011.

[robomics]

uhm... It looks like the thrift package for v0.20.0 is missing (at least for Linux GCC).
Would you mind looking into this @AbrilRBS?

[perseoGI]

@robomics building missing binaries now...

[robomics]

Thanks @perseoGI!
CI seems to be green now.

Can you please review this (and also #24934, #24935, and #24936 if appropriate)?

[perseoGI]

Thanks @perseoGI! CI seems to be green now.

Can you please review this (and also #24934, #24935, and #24936 if appropriate)?

Could you refresh a lit my memory?
Those PRs are pointing to this one but in this PR we are not aiming to bump xsimd

[robomics]

@perseoGI I am bumping xsimd here and in the PRs linked in my other post to address the conflict mentioned in #24382 (review).

[perseoGI]

LGTM

[conan-center-bot]

Conan v1 pipeline ✔️

Warning

Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement.

All green in build 23 (90ef254773c7a0b541c2a0d93256f3bc1a5240bf):

• arrow/17.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/12.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/14.0.2:
  Built 18 packages out of 22 (All logs)

• arrow/16.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/14.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/15.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/16.1.0:
  Built 18 packages out of 22 (All logs)

• arrow/11.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/10.0.1:
  Built 18 packages out of 22 (All logs)

• arrow/13.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/14.0.1:
  Built 18 packages out of 22 (All logs)

• arrow/12.0.1:
  Built 18 packages out of 22 (All logs)

• arrow/8.0.0:
  All packages built successfully! (All logs)

• arrow/8.0.1:
  All packages built successfully! (All logs)

• arrow/10.0.0:
  Built 18 packages out of 22 (All logs)

• arrow/7.0.0:
  All packages built successfully! (All logs)

---

Conan v2 pipeline ✔️

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

All green in build 25 (90ef254773c7a0b541c2a0d93256f3bc1a5240bf):

• arrow/16.1.0:
  All packages built successfully! (All logs)

• arrow/16.0.0:
  All packages built successfully! (All logs)

• arrow/14.0.2:
  All packages built successfully! (All logs)

• arrow/14.0.0:
  All packages built successfully! (All logs)

• arrow/7.0.0:
  Built 6 packages out of 10 (All logs)

• arrow/12.0.1:
  All packages built successfully! (All logs)

• arrow/17.0.0:
  All packages built successfully! (All logs)

• arrow/8.0.0:
  All packages built successfully! (All logs)

• arrow/14.0.1:
  All packages built successfully! (All logs)

• arrow/15.0.0:
  All packages built successfully! (All logs)

• arrow/13.0.0:
  All packages built successfully! (All logs)

• arrow/12.0.0:
  All packages built successfully! (All logs)

• arrow/11.0.0:
  All packages built successfully! (All logs)

• arrow/10.0.0:
  All packages built successfully! (All logs)

• arrow/10.0.1:
  All packages built successfully! (All logs)

• arrow/8.0.1:
  All packages built successfully! (All logs)

[robomics]

@perseoGI CI is green! :)