Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-15 Thread via GitHub


conbench-apache-arrow[bot] commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2975186633

   After merging your PR, Conbench analyzed the 3 benchmarking runs that have 
been run so far on merge-commit 196cde38c112d32a944afe978b6da9c7ce935ef7.
   
   There were 122 benchmark results with an error:
   
   - Commit Run on `arm64-t4g-2xlarge-linux` at [2025-06-15 
21:59:23Z](https://conbench.ursa.dev/compare/runs/c6d836d797954f1e8d014f8ef5c9f2f0...0c30decc03b14bc7bc2f7d58242added/)
 - [`tpch` (R) with engine=arrow, format=parquet, language=R, 
memory_map=False, query_id=TPCH-07, 
scale_factor=1](https://conbench.ursa.dev/benchmark-results/0684f534c0967688800089b9bf4504e2)
 - [`tpch` (R) with engine=arrow, format=native, language=R, 
memory_map=False, query_id=TPCH-09, 
scale_factor=1](https://conbench.ursa.dev/benchmark-results/0684f540203d7d028000e752a0871a05)
   - and 120 more (see the report linked below)
   
   There were no benchmark performance regressions. 🎉
   
   The [full Conbench report](https://github.com/apache/arrow/runs/44147022025) 
has more details.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-15 Thread via GitHub


kou merged PR #46758:
URL: https://github.com/apache/arrow/pull/46758


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-13 Thread via GitHub


hiroyuki-sato commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2972199117

   @assignUser Thanks. It seems that the PR for 20.0.0 is not merge yet.
   I will update this PR if this the PR for 20.0 merged before this PR merge. 
   Otherwise, I'll create new one later.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-13 Thread via GitHub


assignUser commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2971509539

   There are some additional changes in the PR for 20.0.0: 
https://github.com/conan-io/conan-center-index/pull/27311
   Maybe it makes sense to add them too?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-12 Thread via GitHub


kou commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2965137355

   Could you fix lint errors?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-12 Thread via GitHub


kou commented on code in PR #46758:
URL: https://github.com/apache/arrow/pull/46758#discussion_r2141725511


##
ci/conan/all/conanfile.py:
##
@@ -19,6 +19,7 @@
 # LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 # OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
 # SOFTWARE.
+import os

Review Comment:
   ```suggestion
   
   import os
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-11 Thread via GitHub


hiroyuki-sato commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2965242557

   I added `Copyright (c) 2025 Conan.io` as copyright year. Because this file 
was created Fed/2025 
   
https://github.com/conan-io/conan-center-index/commits/master/recipes/arrow/all/patches/19.0.1-0001-fix-cmake.patch
   
   
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


hiroyuki-sato commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957468897

   ```
   ./merge_upstream.sh /tmp/conan-center-index/
   patching file 'all/conandata.yml'
   patching file 'all/conanfile.py'
   1 out of 10 hunks failed--saving rejects to 'all/conanfile.py.rej'
   patching file 'all/patches/19.0.1-0001-fix-cmake.patch'
   patching file 'all/patches/19.0.1-0002-fix-downloaded-mimalloc.patch'
   patching file 'all/test_package/CMakeLists.txt'
   1 out of 1 hunks failed--saving rejects to 
'all/test_package/CMakeLists.txt.rej'
   patching file config.yml
   ```
   
   
   `git diff  
a9b270f9d2052e193ce3c0a6c4e2fda0b0ac5ade..1729c3c2c3b0e9d058821fa00e8a54154415efc6
 recipes/arrow/`
   
   
   ```diff
   diff --git a/recipes/arrow/all/conandata.yml 
b/recipes/arrow/all/conandata.yml
   index f979a4480..a370e59e0 100644
   --- a/recipes/arrow/all/conandata.yml
   +++ b/recipes/arrow/all/conandata.yml
   @@ -1,4 +1,7 @@
sources:
   +  "19.0.1":
   +url: 
"https://www.apache.org/dyn/closer.lua/arrow/arrow-19.0.1/apache-arrow-19.0.1.tar.gz?action=download";
   +sha256: 
"acb76266e8b0c2fbb7eb15d542fbb462a73b3fd1e32b80fad6c2fafd95a51160"
  "18.1.0":
url: 
"https://www.apache.org/dyn/closer.lua/arrow/arrow-18.1.0/apache-arrow-18.1.0.tar.gz?action=download";
sha256: 
"2dc8da5f8796afe213ecc5e5aba85bb82d91520eff3cf315784a52d0fa61d7fc"
   @@ -18,6 +21,13 @@ sources:
url: 
"https://www.apache.org/dyn/closer.lua/arrow/arrow-14.0.2/apache-arrow-14.0.2.tar.gz?action=download";
sha256: 
"1304dedb41896008b89fe0738c71a95d9b81752efc77fa70f264cb1da15d9bc2"
patches:
   +  "19.0.1":
   +- patch_file: "patches/19.0.1-0001-fix-cmake.patch"
   +  patch_description: "use cci package"
   +  patch_type: "conan"
   +- patch_file: "patches/19.0.1-0002-fix-downloaded-mimalloc.patch"
   +  patch_description: "use cci package"
   +  patch_type: "conan"
  "18.1.0":
- patch_file: "patches/18.0.0-0001-fix-cmake.patch"
  patch_description: "use cci package"
   diff --git a/recipes/arrow/all/conanfile.py b/recipes/arrow/all/conanfile.py
   index be8bacf5a..7aef9d284 100644
   --- a/recipes/arrow/all/conanfile.py
   +++ b/recipes/arrow/all/conanfile.py
   @@ -1,3 +1,5 @@
   +import os
   +
from conan import ConanFile
from conan.errors import ConanInvalidConfiguration, ConanException
from conan.tools.build import check_min_cppstd, cross_building
   @@ -6,11 +8,9 @@ from conan.tools.files import apply_conandata_patches, 
copy, export_conandata_pa
from conan.tools.microsoft import is_msvc, is_msvc_static_runtime
from conan.tools.scm import Version

   -import os
   -import glob
   -
required_conan_version = ">=2.1.0"

   +
class ArrowConan(ConanFile):
name = "arrow"
description = "Apache Arrow is a cross-language development platform 
for in-memory data"
   @@ -80,7 +80,7 @@ class ArrowConan(ConanFile):
"dataset_modules": False,
"deprecated": True,
"encryption": False,
   -"filesystem_layer": False,
   +"filesystem_layer": True,
"hdfs_bridgs": False,
"plasma": "deprecated",
"simd_level": "default",
   @@ -120,7 +120,7 @@ class ArrowConan(ConanFile):
def _min_cppstd(self):
# arrow >= 10.0.0 requires C++17.
# https://github.com/apache/arrow/pull/13991
   -return "11" if Version(self.version) < "10.0.0" else "17"
   +return "17"

def export_sources(self):
export_conandata_patches(self)
   @@ -129,10 +129,10 @@ class ArrowConan(ConanFile):
def config_options(self):
if self.settings.os == "Windows":
del self.options.fPIC
   -if Version(self.version) < "8.0.0":
   -del self.options.substrait
if is_msvc(self):
self.options.with_boost = True
   +if Version(self.version) >= "19.0.0":
   +self.options.with_mimalloc = True

def configure(self):
if self.options.shared:
   @@ -187,9 +187,6 @@ class ArrowConan(ConanFile):
self.requires("snappy/1.1.9")
if self.options.get_safe("simd_level") != None or \
self.options.get_safe("runtime_simd_level") != None:
   -if Version(self.version) < 8:
   -self.requires("xsimd/9.0.1")
   -else:
self.requires("xsimd/13.0.0")
if self.options.with_zlib:
self.requires("zlib/[>=1.2.11 <2]")
   @@ -231,15 +228,6 @@ class ArrowConan(ConanFile):
if self.settings.compiler.get_safe("cppstd"):
check_min_cppstd(self, self._min_cppstd)

   -if (
   -Version(self.version) < "10.0.0"
   -and self.settings.compiler == "clang"
   -and Version(self.settings.compil

Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


hiroyuki-sato commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957501875

   @github-actions crossbow submit -g conan


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


hiroyuki-sato commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957527482

   @github-actions crossbow submit -g conan


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


github-actions[bot] commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957530211

   Revision: e8680a0eae6ce788be5d91de41d52335b223f74d
   
   Submitted crossbow builds: [ursacomputing/crossbow @ 
actions-d6d42a5019](https://github.com/ursacomputing/crossbow/branches/all?query=actions-d6d42a5019)
   
   |Task|Status|
   ||--|
   |conan-maximum|[![GitHub 
Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-d6d42a5019-github-conan-maximum)](https://github.com/ursacomputing/crossbow/actions/runs/15549377015/job/43776891868)|
   |conan-minimum|[![GitHub 
Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-d6d42a5019-github-conan-minimum)](https://github.com/ursacomputing/crossbow/actions/runs/15549377147/job/43776892269)|


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


github-actions[bot] commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957504416

   Revision: ce4e7fb3233eb9537cbbacdb57edfb5f858aed02
   
   Submitted crossbow builds: [ursacomputing/crossbow @ 
actions-fa68a234cc](https://github.com/ursacomputing/crossbow/branches/all?query=actions-fa68a234cc)
   
   |Task|Status|
   ||--|
   |conan-maximum|[![GitHub 
Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-fa68a234cc-github-conan-maximum)](https://github.com/ursacomputing/crossbow/actions/runs/15549104346/job/43776130975)|
   |conan-minimum|[![GitHub 
Actions](https://github.com/ursacomputing/crossbow/actions/workflows/crossbow.yml/badge.svg?branch=actions-fa68a234cc-github-conan-minimum)](https://github.com/ursacomputing/crossbow/actions/runs/15549104345/job/43776130977)|


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



Re: [PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


github-actions[bot] commented on PR #46758:
URL: https://github.com/apache/arrow/pull/46758#issuecomment-2957468346

   :warning: GitHub issue #46757 **has been automatically assigned in GitHub** 
to PR creator.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



[PR] GH-46757: [CI][Packaging][Conan] Synchronize upstream conan [arrow]

2025-06-09 Thread via GitHub


hiroyuki-sato opened a new pull request, #46758:
URL: https://github.com/apache/arrow/pull/46758

   ### Rationale for this change
   
   Apache Arrow 19.0.1 available in the conan repository. Need synchronize 
upstream.
   
   ### What changes are included in this PR?
   
   * execute `ci/conan/merge_upstream.sh`
   * Resolve merge failed. (Both files contains license headers. So patch 
failed)
 * `ci/conan/all/test_package/CMakeLists.txtj`
 * `ci/conan/all/conanfile.py`
   
   ### Are these changes tested?
   
   Yes.
   
   ### Are there any user-facing changes?
   
   No.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org