llama90 commented on code in PR #41378:
URL: https://github.com/apache/arrow/pull/41378#discussion_r1581769431
##########
.github/workflows/python.yml:
##########
@@ -171,6 +171,23 @@ jobs:
with:
fetch-depth: 0
submodules: recursive
+ - name: Install ccache
Review Comment:
Thank you for the review.
How long will it take for an alternative solution to be allowed?
As you said, we understood that the cache was not configured properly
because the same key was used on both platforms, and we fixed this.
#### AMD64 macOS 12 Python 3
https://github.com/apache/arrow/actions/runs/8858657235/job/24327563485?pr=41378#step:11:42
#### ARM64 macOS 14 Python 3
https://github.com/apache/arrow/actions/runs/8858657235/job/24327563591?pr=41378#step:11:42
Furthermore, we understand that the cache capacity is fundamentally limited
and the cache is removed frequently, making it not used efficiently. If what
you mentioned is applied, I expect that overall CI time will be reduced.
And until an alternative solution is applied, would it be better to proceed
with the PR as a draft?
--
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]
