Skip to content

Commit 72bb3e3

Browse files
paleolimbotCopilot
andauthored
GH-49744: [Integration][Format] Remove unneeded skips for Rust and nanoarrow integration tests (#49910)
### Rationale for this change The current skip list in the Archery datagen (where the skip list is kept) is out of date. ### What changes are included in this PR? Skips were removed. ### Are these changes tested? Yes, the integration test CI run should ensure these skips can in fact be removed. ### Are there any user-facing changes? No * GitHub Issue: #49744 --------- Co-authored-by: Copilot <copilot@github.com>
1 parent 6d6f246 commit 72bb3e3

2 files changed

Lines changed: 6 additions & 10 deletions

File tree

.github/workflows/integration.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -137,6 +137,7 @@ jobs:
137137
-e ARCHERY_INTEGRATION_WITH_JS=1 \
138138
-e ARCHERY_INTEGRATION_WITH_NANOARROW=1 \
139139
-e ARCHERY_INTEGRATION_WITH_RUST=1 \
140+
-e RUST_BACKTRACE=1 \
140141
conda-integration
141142
- name: Docker Push
142143
if: >-

dev/archery/archery/integration/datagen.py

Lines changed: 5 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1936,17 +1936,13 @@ def get_generated_json_files(tempdir=None):
19361936
generate_decimal32_case()
19371937
.skip_tester('Java')
19381938
.skip_tester('JS')
1939-
.skip_tester('nanoarrow')
19401939
.skip_tester('Ruby')
1941-
.skip_tester('Rust')
19421940
.skip_tester('Go'),
19431941

19441942
generate_decimal64_case()
19451943
.skip_tester('Java')
19461944
.skip_tester('JS')
1947-
.skip_tester('nanoarrow')
19481945
.skip_tester('Ruby')
1949-
.skip_tester('Rust')
19501946
.skip_tester('Go'),
19511947

19521948
generate_datetime_case(),
@@ -2003,25 +1999,24 @@ def get_generated_json_files(tempdir=None):
20031999
.skip_tester('JS')
20042000
# TODO(https://github.com/apache/arrow-nanoarrow/issues/618)
20052001
.skip_tester('nanoarrow')
2006-
.skip_tester('Ruby')
2007-
.skip_tester('Rust'),
2002+
.skip_tester('Ruby'),
20082003

20092004
generate_binary_view_case()
20102005
.skip_tester('JS')
20112006
# TODO(https://github.com/apache/arrow-nanoarrow/issues/618)
20122007
.skip_tester('nanoarrow')
2013-
.skip_tester('Ruby')
2014-
.skip_tester('Rust'),
2008+
.skip_tester('Ruby'),
20152009

20162010
generate_list_view_case()
20172011
.skip_tester('.NET') # Doesn't support large list views
20182012
.skip_tester('JS')
20192013
# TODO(https://github.com/apache/arrow-nanoarrow/issues/618)
20202014
.skip_tester('nanoarrow')
2021-
.skip_tester('Ruby')
2022-
.skip_tester('Rust'),
2015+
.skip_tester('Ruby'),
20232016

20242017
generate_extension_case()
2018+
# Also contains a dictionary column
2019+
# TODO(https://github.com/apache/arrow-nanoarrow/issues/622)
20252020
.skip_tester('nanoarrow')
20262021
# TODO(https://github.com/apache/arrow/issues/38045)
20272022
.skip_format(SKIP_FLIGHT, '.NET')

0 commit comments

Comments
 (0)