Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 #343

Merged
merged 1 commit into from
Feb 13, 2024

Conversation

amoeba
Copy link
Member

@amoeba amoeba commented Feb 1, 2024

Can't easily test this locally so sending a PR to see what CI looks like. Java changes are in a separate PR: #339.

@raulcd

@amoeba amoeba changed the title [MINOR][RELEASE] Update C++ and PyArrow versions for 15.0.0 MINOR: [RELEASE] Update C++ and PyArrow versions for 15.0.0 Feb 1, 2024
@amoeba amoeba changed the title MINOR: [RELEASE] Update C++ and PyArrow versions for 15.0.0 MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 Feb 1, 2024
python/requirements.txt Outdated Show resolved Hide resolved
@raulcd
Copy link
Member

raulcd commented Feb 1, 2024

Sorry, I forgot to run the release update for the cookbooks for the 14.0.x patch releases and 15.0.0, did you use the 01-bump-versions.sh 14.0.0 15.0.0 script?
https://github.com/apache/arrow-cookbook#usage

@amoeba
Copy link
Member Author

amoeba commented Feb 1, 2024

I didn't originally but did just now (sh dev/release/01-bump-versions.sh 14.0.0 15.0.0) and squashed everything into a single commit. I'll do that in the future.

@amoeba
Copy link
Member Author

amoeba commented Feb 1, 2024

@davisusanibar can you look at the Java CI failures?

@davisusanibar
Copy link
Contributor

@davisusanibar can you look at the Java CI failures?

Let me take a look

@davisusanibar
Copy link
Contributor

@davisusanibar can you look at the Java CI failures?

Let me take a look

A Java nightly is trying to download artifacts from https://nightlies.apache.org/arrow/java, but org.apache.arrow:arrow-memory-netty-buffer-patch:jar:16.0.0-SNAPSHOT is not being uploaded.

This issue has been created to track this: #342. Let me check the activities needed to upload that to the nightly repository.

@davisusanibar
Copy link
Contributor

CC @vibhatha @danepitkin

@davisusanibar
Copy link
Contributor

PR is ready for review: apache/arrow#39901

@amoeba
Copy link
Member Author

amoeba commented Feb 6, 2024

@raulcd I think now that apache/arrow#39901 the Java checks should pass, can you re-run them to check?

@raulcd
Copy link
Member

raulcd commented Feb 7, 2024

There seems to still be some errors:

 SLF4J(W): See https://www.slf4j.org/codes.html#ignoredBindings for an explanation.
Exception java.lang.UnsatisfiedLinkError: /tmp/jnilib-9784155468287532443.tmp: /tmp/jnilib-9784155468287532443.tmp: undefined symbol: _ZTIN6google8protobuf7MessageE

I can't spend much time diagnosing at the moment, @davisusanibar any idea what is happening on the Java failure?

@davisusanibar
Copy link
Contributor

There seems to still be some errors:

 SLF4J(W): See https://www.slf4j.org/codes.html#ignoredBindings for an explanation.
Exception java.lang.UnsatisfiedLinkError: /tmp/jnilib-9784155468287532443.tmp: /tmp/jnilib-9784155468287532443.tmp: undefined symbol: _ZTIN6google8protobuf7MessageE

I can't spend much time diagnosing at the moment, @davisusanibar any idea what is happening on the Java failure?

I am reviewing that . More details at: #339 (comment)

Copy link
Contributor

@vibhatha vibhatha left a comment

Choose a reason for hiding this comment

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

@davisusanibar can we check whether the Java-Jar related scenario is resolved?

@vibhatha
Copy link
Contributor

We should probably re-run the CIs after merging apache/arrow#40015

@davisusanibar
Copy link
Contributor

We should probably re-run the CIs after merging apache/arrow#40015

This PR depend of #339

@davisusanibar
Copy link
Contributor

We should probably re-run the CIs after merging apache/arrow#40015

This PR depend of #339

@amoeba It's ready for a rebase

@amoeba
Copy link
Member Author

amoeba commented Feb 12, 2024

Thanks @davisusanibar. I rebased.

@amoeba
Copy link
Member Author

amoeba commented Feb 12, 2024

Hi @raulcd, I think this is ready for a merge. The CI passes here and the Java CI is passing as of https://github.com/apache/arrow-cookbook/actions/runs/7865593306. Can you have a look?

Copy link
Member

@raulcd raulcd left a comment

Choose a reason for hiding this comment

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

Thanks for following this up!

@raulcd raulcd merged commit 440f5e0 into apache:main Feb 13, 2024
2 checks passed
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.

5 participants