Skip to content

Commit bdeb4ae

Browse files
kevjumbakevjumba
authored
fix: Fix build wheels workflow to install apache-arrow correctly (#2932)
* Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai> * Fix Signed-off-by: Kevin Zhang <kzhang@tecton.ai>
1 parent 292adc2 commit bdeb4ae

File tree

2 files changed

+15
-12
lines changed

2 files changed

+15
-12
lines changed

‎.github/workflows/build_wheels.yml‎

Lines changed: 6 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -62,18 +62,6 @@ jobs:
6262
registry-url: 'https://registry.npmjs.org'
6363
- name: Build UI
6464
run: make build-ui
65-
- name: Install apache-arrow on ubuntu
66-
if: matrix.os == 'ubuntu-latest'
67-
run: |
68-
sudo apt update
69-
sudo apt install -y -V ca-certificates lsb-release wget
70-
wget https://apache.jfrog.io/artifactory/arrow/$(lsb_release --id --short | tr 'A-Z' 'a-z')/apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb
71-
sudo apt install -y -V ./apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb
72-
sudo apt update
73-
sudo apt install -y -V libarrow-dev
74-
- name: Install apache-arrow on macos
75-
if: matrix.os == 'macOS-latest'
76-
run: brew install apache-arrow
7765
- name: Build wheels
7866
uses: pypa/cibuildwheel@v2.7.0
7967
env:
@@ -86,7 +74,12 @@ jobs:
8674
curl -o go.tar.gz https://dl.google.com/go/go1.18.2.linux-amd64.tar.gz
8775
tar -C /usr/local -xzf go.tar.gz
8876
go version
77+
yum -y update &&
78+
yum install -y epel-release || yum install -y https://dl.fedoraproject.org/pub/epel/epel-release-latest-$(cut -d: -f5 /etc/system-release-cpe | cut -d. -f1).noarch.rpm &&
79+
yum install -y https://apache.jfrog.io/artifactory/arrow/centos/$(cut -d: -f5 /etc/system-release-cpe | cut -d. -f1)/apache-arrow-release-latest.rpm &&
80+
yum install -y --enablerepo=epel arrow-devel # For C++
8981
CIBW_BEFORE_ALL_MACOS: |
82+
brew install apache-arrow
9083
curl -o python.pkg https://www.python.org/ftp/python/3.9.12/python-3.9.12-macosx10.9.pkg
9184
sudo installer -pkg python.pkg -target /
9285
# There's a `git restore` in here because `make install-go-ci-dependencies` is actually messing up go.mod & go.sum.
@@ -111,6 +104,7 @@ jobs:
111104
CIBW_BEFORE_BUILD: |
112105
git status
113106
git restore go.mod go.sum
107+
brew install apache-arrow
114108
- uses: actions/upload-artifact@v2
115109
with:
116110
name: wheels

‎.github/workflows/pr_local_integration_tests.yml‎

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,15 @@ jobs:
5656
${{ runner.os }}-${{ steps.setup-python.outputs.python-version }}-pip-
5757
- name: Install pip-tools
5858
run: pip install pip-tools
59+
- name: Install apache-arrow on ubuntu
60+
if: matrix.os == 'ubuntu-latest'
61+
run: |
62+
sudo apt update
63+
sudo apt install -y -V ca-certificates lsb-release wget
64+
wget https://apache.jfrog.io/artifactory/arrow/$(lsb_release --id --short | tr 'A-Z' 'a-z')/apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb
65+
sudo apt install -y -V ./apache-arrow-apt-source-latest-$(lsb_release --codename --short).deb
66+
sudo apt update
67+
sudo apt install -y -V libarrow-dev
5968
- name: Install dependencies
6069
run: make install-python-ci-dependencies
6170
- name: Test local integration tests

0 commit comments

Comments
 (0)