Skip to content

Commit

Permalink
ARROW-16636: [Rust] Activate several IPC integration tests for rust (#…
Browse files Browse the repository at this point in the history
…13219)

arrow-rs has fixed severals integration test failures (apache/arrow-rs#1404):

generate_decimal128_case
generate_interval_case
generate_map_case
generate_non_canonical_map_case
generate_nested_large_offsets_case
generate_nested_dictionary_case
generate_unions_case

And this one passes test without any fix:
generate_extension_case

We should activate these IPC integration tests for rust.

Authored-by: Liang-Chi Hsieh <viirya@gmail.com>
Signed-off-by: Andrew Lamb <andrew@nerdnetworks.org>
  • Loading branch information
viirya committed Jun 11, 2022
1 parent 44eb260 commit 42a2efd
Showing 1 changed file with 8 additions and 16 deletions.
24 changes: 8 additions & 16 deletions dev/archery/archery/integration/datagen.py
Expand Up @@ -1598,8 +1598,7 @@ def _temp_path():
.skip_category('C#')
.skip_category('JS'), # TODO(ARROW-7900)

generate_decimal128_case()
.skip_category('Rust'),
generate_decimal128_case(),

generate_decimal256_case()
.skip_category('Go') # TODO(ARROW-7948): Decimal + Go
Expand All @@ -1615,22 +1614,19 @@ def _temp_path():

generate_interval_case()
.skip_category('C#')
.skip_category('JS') # TODO(ARROW-5239): Intervals + JS
.skip_category('Rust'),
.skip_category('JS'), # TODO(ARROW-5239): Intervals + JS

generate_month_day_nano_interval_case()
.skip_category('C#')
.skip_category('JS'),

generate_map_case()
.skip_category('C#')
.skip_category('Rust'),
.skip_category('C#'),

generate_non_canonical_map_case()
.skip_category('C#')
.skip_category('Java') # TODO(ARROW-8715)
.skip_category('JS') # TODO(ARROW-8716)
.skip_category('Rust'),
.skip_category('JS'), # TODO(ARROW-8716)

generate_nested_case()
.skip_category('C#'),
Expand All @@ -1641,14 +1637,12 @@ def _temp_path():
generate_nested_large_offsets_case()
.skip_category('C#')
.skip_category('Go')
.skip_category('JS')
.skip_category('Rust'),
.skip_category('JS'),

generate_unions_case()
.skip_category('C#')
.skip_category('Go')
.skip_category('JS')
.skip_category('Rust'),
.skip_category('JS'),

generate_custom_metadata_case()
.skip_category('C#')
Expand All @@ -1670,13 +1664,11 @@ def _temp_path():
generate_nested_dictionary_case()
.skip_category('C#')
.skip_category('Java') # TODO(ARROW-7779)
.skip_category('JS')
.skip_category('Rust'),
.skip_category('JS'),

generate_extension_case()
.skip_category('C#')
.skip_category('JS')
.skip_category('Rust'),
.skip_category('JS'),
]

generated_paths = []
Expand Down

0 comments on commit 42a2efd

Please sign in to comment.