Skip to content

Commit

Permalink
GH-623: Cleanup enabling Gandiva tests change (#624)
Browse files Browse the repository at this point in the history
## What's Changed

* Add a comment when we can remove this workaround.
* Remove a needless workaround for Windows.

Closes #623.
  • Loading branch information
kou authored Feb 20, 2025
1 parent 81a5c2d commit 799d9fd
Showing 1 changed file with 12 additions and 6 deletions.
18 changes: 12 additions & 6 deletions .github/workflows/rc.yml
Original file line number Diff line number Diff line change
Expand Up @@ -107,6 +107,12 @@ jobs:
- name: Extract source archive
run: |
tar -xf apache-arrow-java-*.tar.gz --strip-components=1
# We always use the main branch for apache/arrow for now.
# Because we want to use
# https://github.com/apache/arrow/pull/45114 in
# apache/arrow-java. We can revert this workaround once Apache
# Arrow 20.0.0 that includes the change released.
#
# - name: Download the latest Apache Arrow C++
# if: github.event_name != 'schedule'
# run: |
Expand Down Expand Up @@ -299,13 +305,13 @@ jobs:
shell: bash
run: |
tar -xf apache-arrow-java-*.tar.gz --strip-components=1
# - name: Download the latest Apache Arrow C++
# if: github.event_name != 'schedule'
# shell: bash
# run: |
# ci/scripts/download_cpp.sh
- name: Download the latest Apache Arrow C++
if: github.event_name != 'schedule'
shell: bash
run: |
ci/scripts/download_cpp.sh
- name: Checkout Apache Arrow C++
# if: github.event_name == 'schedule'
if: github.event_name == 'schedule'
uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
with:
repository: apache/arrow
Expand Down

0 comments on commit 799d9fd

Please sign in to comment.