Skip to content

[main] Rebuild for libabseil20230125#975

Merged
h-vetinari merged 6 commits intoconda-forge:mainfrom
h-vetinari:abseil
Mar 4, 2023
Merged

[main] Rebuild for libabseil20230125#975
h-vetinari merged 6 commits intoconda-forge:mainfrom
h-vetinari:abseil

Conversation

@h-vetinari
Copy link
Copy Markdown
Member

Bot stuck on tensorflow

@conda-forge-webservices
Copy link
Copy Markdown

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.

@h-vetinari h-vetinari added the automerge Merge the PR when CI passes label Feb 24, 2023
@github-actions
Copy link
Copy Markdown
Contributor

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

  • linter: passed
  • azure: failed

Thus the PR was not passing and not merged.

@h-vetinari
Copy link
Copy Markdown
Member Author

It seems this is segfaulting in test_flight.py::test_flight_list_flights on osx-64 both for arrow 10 & 11, might be related to conda-forge/grpc-cpp-feedstock#281, which I haven't had time to look into yet...

CC @xhochy @kkraus14 @kou @pitrou @coryan

@h-vetinari
Copy link
Copy Markdown
Member Author

It seems this is segfaulting in test_flight.py::test_flight_list_flights on osx-64 both for arrow 10 & 11

Looking at the source of that test and the utility class ConstantFlightServer, it notes:

See ARROW-4796: this server implementation will segfault if Flight
does not properly hold a reference to the Table object.

@h-vetinari
Copy link
Copy Markdown
Member Author

@kou @pitrou
Any idea what might be causing these segfaults on osx? This is now holding up other migrations like re2... 😢

@h-vetinari h-vetinari removed the automerge Merge the PR when CI passes label Mar 1, 2023
@h-vetinari
Copy link
Copy Markdown
Member Author

@conda-forge/arrow-cpp @kou @pitrou @jorisvandenbossche @kkraus14

A lot of migrations are starting to pile up on the missing abseil migration for arrow 10/11. I'm going to skip the flight tests on osx now to unblock this. Any input/help is welcome.

@kkraus14
Copy link
Copy Markdown
Contributor

kkraus14 commented Mar 3, 2023

@conda-forge/arrow-cpp @kou @pitrou @jorisvandenbossche @kkraus14

A lot of migrations are starting to pile up on the missing abseil migration for arrow 10/11. I'm going to skip the flight tests on osx now to unblock this. Any input/help is welcome.

I'll ping a few folks internally to see if I can get someone to jump in here.

@h-vetinari
Copy link
Copy Markdown
Member Author

I'll ping a few folks internally to see if I can get someone to jump in here.

@kkraus14, not sure what ever came of this, but there's now an upstream issue to track this: apache/arrow#36908

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants