You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by "raulcd (via GitHub)" <gi...@apache.org> on 2023/10/26 11:16:06 UTC

Re: [I] [CI][Release] jpype install is missing in verify-release-candidate.sh [arrow]

raulcd commented on issue #38470:
URL: https://github.com/apache/arrow/issues/38470#issuecomment-1780914779

   I think we should install jpype1 on the verification script too:
   https://github.com/apache/arrow/blob/main/dev/release/verify-release-candidate.sh#L906
   Same as we do here:
   https://github.com/apache/arrow/pull/38248/files#diff-541c9c4d96250540b1d053548895b96e15c74001c00102b0c0c3686d8b62c54dR27
   
   @pitrou is there any reason why we don't install those as part of `archery[integration]`?
   https://github.com/apache/arrow/blob/main/dev/archery/setup.py#L37
   
   Or is it ok if I just add them there?


-- 
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