Skip to content

Commit 616183f

Browse files
committed
Apply CI workflow and test fixes from PR #745
- Update actions/checkout@v2 → v4 and actions/setup-python@v2 → v5 in run-unit-tests-with-arrow job - Pin Poetry version to 2.2.1 across all workflow files - Reduce long-running query test min duration from 3 to 2 minutes Co-authored-by: Isaac
1 parent 945c14a commit 616183f

File tree

5 files changed

+12
-5
lines changed

5 files changed

+12
-5
lines changed

.github/workflows/code-quality-checks.yml

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ jobs:
3535
- name: Install Poetry
3636
uses: snok/install-poetry@v1
3737
with:
38+
version: "2.2.1"
3839
virtualenvs-create: true
3940
virtualenvs-in-project: true
4041
installer-parallel: true
@@ -106,10 +107,10 @@ jobs:
106107
# check-out repo and set-up python
107108
#----------------------------------------------
108109
- name: Check out repository
109-
uses: actions/checkout@v2
110+
uses: actions/checkout@v4
110111
- name: Set up python ${{ matrix.python-version }}
111112
id: setup-python
112-
uses: actions/setup-python@v2
113+
uses: actions/setup-python@v5
113114
with:
114115
python-version: ${{ matrix.python-version }}
115116
#----------------------------------------------
@@ -118,6 +119,7 @@ jobs:
118119
- name: Install Poetry
119120
uses: snok/install-poetry@v1
120121
with:
122+
version: "2.2.1"
121123
virtualenvs-create: true
122124
virtualenvs-in-project: true
123125
installer-parallel: true
@@ -191,6 +193,7 @@ jobs:
191193
- name: Install Poetry
192194
uses: snok/install-poetry@v1
193195
with:
196+
version: "2.2.1"
194197
virtualenvs-create: true
195198
virtualenvs-in-project: true
196199
installer-parallel: true
@@ -243,6 +246,7 @@ jobs:
243246
- name: Install Poetry
244247
uses: snok/install-poetry@v1
245248
with:
249+
version: "2.2.1"
246250
virtualenvs-create: true
247251
virtualenvs-in-project: true
248252
installer-parallel: true

.github/workflows/integration.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -33,6 +33,7 @@ jobs:
3333
- name: Install Poetry
3434
uses: snok/install-poetry@v1
3535
with:
36+
version: "2.2.1"
3637
virtualenvs-create: true
3738
virtualenvs-in-project: true
3839
installer-parallel: true

.github/workflows/publish-test.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ jobs:
2121
- name: Install Poetry
2222
uses: snok/install-poetry@v1
2323
with:
24+
version: "2.2.1"
2425
virtualenvs-create: true
2526
virtualenvs-in-project: true
2627
installer-parallel: true

.github/workflows/publish.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ jobs:
2323
- name: Install Poetry
2424
uses: snok/install-poetry@v1
2525
with:
26+
version: "2.2.1"
2627
virtualenvs-create: true
2728
virtualenvs-in-project: true
2829
installer-parallel: true

tests/e2e/common/large_queries_mixin.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -106,11 +106,11 @@ def test_query_with_large_narrow_result_set(self, extra_params):
106106
],
107107
)
108108
def test_long_running_query(self, extra_params):
109-
"""Incrementally increase query size until it takes at least 3 minutes,
109+
"""Incrementally increase query size until it takes at least 2 minutes,
110110
and asserts that the query completes successfully.
111111
"""
112112
minutes = 60
113-
min_duration = 3 * minutes
113+
min_duration = 2 * minutes
114114

115115
duration = -1
116116
scale0 = 10000
@@ -136,5 +136,5 @@ def test_long_running_query(self, extra_params):
136136
duration = time.time() - start
137137
current_fraction = duration / min_duration
138138
print("Took {} s with scale factor={}".format(duration, scale_factor))
139-
# Extrapolate linearly to reach 3 min and add 50% padding to push over the limit
139+
# Extrapolate linearly to reach 2 min and add 50% padding to push over the limit
140140
scale_factor = math.ceil(1.5 * scale_factor / current_fraction)

0 commit comments

Comments
 (0)