kou commented on code in PR #43385:
URL: https://github.com/apache/arrow/pull/43385#discussion_r1694423646
##########
dev/tasks/java-jars/github.yml:
##########
@@ -140,6 +140,12 @@ jobs:
brew uninstall protobuf
brew bundle --file=arrow/java/Brewfile
+
+ # We want to use the bundled googletest for static linking. Since
+ # both BUNDLED and brew options are enabled, it could cause a
conflict
+ # when there is a version mismatch.
+ # We uninstall googletest to ensure using the bundled googletest.
+ brew uninstall googletest
Review Comment:
Could you move this part before `brew bundle --file=arrow/java/Brewfile`?
Because `googletest` is in `cpp/Brewfile` not `java/Brewfile`.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]