You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by "amoeba (via GitHub)" <gi...@apache.org> on 2024/02/01 06:04:16 UTC

[PR] [MINOR][RELEASE] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

amoeba opened a new pull request, #343:
URL: https://github.com/apache/arrow-cookbook/pull/343

   Can't easily test this locally so sending a PR to see what CI looks like. Java changes are in a separate PR: https://github.com/apache/arrow-cookbook/pull/339.


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1932470360

   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?


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd commented on code in PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#discussion_r1474477916


##########
python/requirements.txt:
##########
@@ -1,5 +1,5 @@
 Sphinx>=4.0.2
-pyarrow==14.0.0
+pyarrow==15.0.0
 pandas>=1.2.5
 opentelemetry-api>=1.0.0
-opentelemetry-sdk>=1.0.0
\ No newline at end of file
+opentelemetry-sdk>=1.0.0opentelemetry-sdk>=1.0.0

Review Comment:
   We do and pip seems to do some magic knowing the name:
   ```
    Collecting opentelemetry-sdk>=1.0.0opentelemetry-sdk>=1.0.0
     Downloading opentelemetry_sdk-1.22.0-py3-none-any.whl (105 kB)
        ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 105.6/105.6 KB 11.3 MB/s eta 0:00:00
   ```



-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1922563885

   PR is ready for review: https://github.com/apache/arrow/pull/39901


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1938782245

   > > We should probably re-run the CIs after merging [apache/arrow#40015](https://github.com/apache/arrow/pull/40015)
   > 
   > This PR depend of #339
   
   @amoeba It's ready for a rebase


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on code in PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#discussion_r1474821790


##########
python/requirements.txt:
##########
@@ -1,5 +1,5 @@
 Sphinx>=4.0.2
-pyarrow==14.0.0
+pyarrow==15.0.0
 pandas>=1.2.5
 opentelemetry-api>=1.0.0
-opentelemetry-sdk>=1.0.0
\ No newline at end of file
+opentelemetry-sdk>=1.0.0opentelemetry-sdk>=1.0.0

Review Comment:
   I have no idea what happened and didn't see this when I sent in the PR. I'll push a fix.



-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "lidavidm (via GitHub)" <gi...@apache.org>.
lidavidm commented on code in PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#discussion_r1474456498


##########
python/requirements.txt:
##########
@@ -1,5 +1,5 @@
 Sphinx>=4.0.2
-pyarrow==14.0.0
+pyarrow==15.0.0
 pandas>=1.2.5
 opentelemetry-api>=1.0.0
-opentelemetry-sdk>=1.0.0
\ No newline at end of file
+opentelemetry-sdk>=1.0.0opentelemetry-sdk>=1.0.0

Review Comment:
   Something got messed up here? Do we even use this file if CI still passes?



-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1922181514

   > @davisusanibar can you look at the Java CI failures?
   
   Let me take a look


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd merged PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "raulcd (via GitHub)" <gi...@apache.org>.
raulcd commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1921339735

   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
   


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1922193807

   > > @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: https://github.com/apache/arrow-cookbook/issues/342. Let me check the activities needed to upload that to the nightly repository.


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on code in PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#discussion_r1474825846


##########
python/requirements.txt:
##########
@@ -1,5 +1,5 @@
 Sphinx>=4.0.2
-pyarrow==14.0.0
+pyarrow==15.0.0
 pandas>=1.2.5
 opentelemetry-api>=1.0.0
-opentelemetry-sdk>=1.0.0
\ No newline at end of file
+opentelemetry-sdk>=1.0.0opentelemetry-sdk>=1.0.0

Review Comment:
   Fixed. Thanks for catching the mistake.



-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1939364790

   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?


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1937934216

   > We should probably re-run the CIs after merging [apache/arrow#40015](https://github.com/apache/arrow/pull/40015)
   
   This PR depend of https://github.com/apache/arrow-cookbook/pull/339 


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "vibhatha (via GitHub)" <gi...@apache.org>.
vibhatha commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1936775816

   We should probably re-run the CIs after merging https://github.com/apache/arrow/pull/40015


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1922020863

   @davisusanibar can you look at the Java CI failures?


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1922197281

   CC @vibhatha @danepitkin 


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1939327273

   Thanks @davisusanibar. I rebased and will keep an eye on CI.


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "davisusanibar (via GitHub)" <gi...@apache.org>.
davisusanibar commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1932495442

   > 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: https://github.com/apache/arrow-cookbook/pull/339#issuecomment-1932429139


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1921991610

   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. 


-- 
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-unsubscribe@arrow.apache.org

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


Re: [PR] MINOR: [Release] Update C++ and PyArrow versions for 15.0.0 [arrow-cookbook]

Posted by "amoeba (via GitHub)" <gi...@apache.org>.
amoeba commented on PR #343:
URL: https://github.com/apache/arrow-cookbook/pull/343#issuecomment-1930460596

   @raulcd I think now that https://github.com/apache/arrow/pull/39901 the Java checks should pass, can you re-run them to check?


-- 
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-unsubscribe@arrow.apache.org

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