Enable test suite of `libarrow` #1058

h-vetinari · 2023-05-15T10:05:16Z

Since #875, we're now running with the test suite for pyarrow. While working on #1048, I wanted to ensure that also the library portion works as intended, so thought about switching on the test suite for libarrow.

After some initial attempts it looks like this is failing, hence will need some iteration together with upstream.

conda-forge-webservices · 2023-05-15T10:05:24Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

conda-forge-webservices · 2024-08-19T03:34:16Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

) ### Rationale for this change Enable tests for libarrow in conda-forge: #35587 ### What changes are included in this PR? old glibc does not actually support timezones like `-0117` (used in `StrptimeZoneOffset` test). The exact lower bound for glibc is hard for me to determine; I know that it passes with 2.28 and that it fails with 2.17. Anything in between is an open question. I went with the conservative option here. ### Are these changes tested? Tested in conda-forge/arrow-cpp-feedstock#1058 ### Are there any user-facing changes? * GitHub Issue: #43808 Lead-authored-by: H. Vetinari <[email protected]> Co-authored-by: Sutou Kouhei <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>

except those depending on timing, following upstream, see https://github.com/apache/arrow/blob/apache-arrow-12.0.0/ci/appveyor-cpp-build.bat#L63-L68

also requires git submodule for parquet test data

conda-forge-admin · 2024-11-23T00:06:26Z

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.
ℹ️ The recipe is not parsable by parser conda-recipe-manager. Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

_{This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12000178104. Examine the logs at this URL for more detail.}

cannot run test suite without an actual GPU

h-vetinari · 2024-11-25T00:24:31Z

Alrighty, after over 2 years of working towards this, we now have the C++ tests running for libarrow 🥳

Enable test suite of `libarrow`

h-vetinari force-pushed the test_lib branch from 08b8a86 to 95c79d4 Compare May 15, 2023 10:09

conda-forge deleted a comment from conda-forge-webservices bot May 15, 2023

h-vetinari force-pushed the test_lib branch from 95c79d4 to 909e751 Compare May 15, 2023 10:13

h-vetinari mentioned this pull request May 15, 2023

[C++] Test failures for libarrow in conda-forge apache/arrow#35587

Closed

h-vetinari force-pushed the test_lib branch from 909e751 to 0c9affe Compare August 24, 2023 08:49

h-vetinari mentioned this pull request Sep 21, 2023

[Python][CI] Compatibility with newer minio apache/arrow#37692

Closed

h-vetinari force-pushed the test_lib branch from 0c9affe to e26333f Compare November 15, 2023 20:04

h-vetinari force-pushed the test_lib branch from e26333f to dc0da85 Compare November 30, 2023 23:29

h-vetinari force-pushed the test_lib branch from 4beee77 to 139f788 Compare June 24, 2024 05:06

h-vetinari force-pushed the test_lib branch from 35e9296 to 2300bbb Compare August 19, 2024 03:34

h-vetinari mentioned this pull request Aug 19, 2024

[C++] Add support for Boost 1.86 apache/arrow#43746

Closed

h-vetinari force-pushed the test_lib branch from 2300bbb to e40aa8e Compare August 19, 2024 04:16

h-vetinari force-pushed the test_lib branch 2 times, most recently from 26074b3 to afb81ea Compare November 2, 2024 04:45

h-vetinari mentioned this pull request Nov 2, 2024

GH-43808: [C++] skip -0117 in StrptimeZoneOffset for old glibc apache/arrow#44621

Merged

h-vetinari added 7 commits November 23, 2024 11:00

also build & run tests for library

a53337a

except those depending on timing, following upstream, see https://github.com/apache/arrow/blob/apache-arrow-12.0.0/ci/appveyor-cpp-build.bat#L63-L68

add test dependencies

650f737

populate {ARROW,PARQUET}_TEST_DATA

5db6498

also requires git submodule for parquet test data

tests seem to need libboost CMake metadata

ffa118a

need minio for s3fs tests

edbf821

install azurite before running tests

7c2623e

add tzdata to build environment

396ac10

h-vetinari force-pushed the test_lib branch from afb81ea to 11a8373 Compare November 23, 2024 00:04

h-vetinari added 3 commits November 23, 2024 16:20

skip StrptimeZoneOffset for old glibc and gcsfs tests everywhere

6fd41d5

update fix for glibc vs tz issue

e5b91bf

skip libarrow tests for CUDA builds

aa58484

cannot run test suite without an actual GPU

h-vetinari force-pushed the test_lib branch from 11a8373 to aa58484 Compare November 23, 2024 05:24

h-vetinari changed the title ~~WIP: Enable test suite of libarrow~~ Enable test suite of libarrow Nov 23, 2024

skip NonExistentBucket test on osx

ba7b678

h-vetinari force-pushed the test_lib branch from 5a8e6f7 to ba7b678 Compare November 24, 2024 22:36

h-vetinari marked this pull request as ready for review November 24, 2024 23:54

h-vetinari requested review from cpcloud, jakirkham, jreback, kou, kszucs, leifwalsh, nealrichardson, pcmoritz, pearu, pitrou, raulcd, robertnishihara, siddharthteotia, wesm and xhochy as code owners November 24, 2024 23:54

h-vetinari added a commit that referenced this pull request Nov 25, 2024

Merge pull request #1058 from h-vetinari/test_lib

efccc47

Enable test suite of `libarrow`

h-vetinari merged commit ba7b678 into conda-forge:main Nov 25, 2024
13 checks passed

h-vetinari deleted the test_lib branch November 25, 2024 00:24

This was referenced Dec 9, 2024

[C++] Allow building against system opentelemetry apache/arrow#44982

Closed

[C++] Provide option to opt out of installing tests apache/arrow#44993

Open

h-vetinari mentioned this pull request Dec 29, 2024

[C++] Failing arrow-flight-test when enabling opentelemetry apache/arrow#45120

Open

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Enable test suite of `libarrow` #1058

Enable test suite of `libarrow` #1058

h-vetinari commented May 15, 2023

conda-forge-webservices bot commented May 15, 2023

conda-forge-webservices bot commented Aug 19, 2024 •

edited by conda-forge-admin

Loading

conda-forge-admin commented Nov 23, 2024 •

edited

Loading

h-vetinari commented Nov 25, 2024

Enable test suite of libarrow #1058

Enable test suite of libarrow #1058

Conversation

h-vetinari commented May 15, 2023

conda-forge-webservices bot commented May 15, 2023

conda-forge-webservices bot commented Aug 19, 2024 • edited by conda-forge-admin Loading

conda-forge-admin commented Nov 23, 2024 • edited Loading

h-vetinari commented Nov 25, 2024

Enable test suite of `libarrow` #1058

Enable test suite of `libarrow` #1058

conda-forge-webservices bot commented Aug 19, 2024 •

edited by conda-forge-admin

Loading

conda-forge-admin commented Nov 23, 2024 •

edited

Loading