paleolimbot commented on code in PR #41765:
URL: https://github.com/apache/arrow/pull/41765#discussion_r2016950647
##########
python/pyarrow/tests/parquet/test_data_types.py:
##########
@@ -521,3 +522,20 @@ def test_json_extension_type(storage_type):
table = pa.table([arr], names=["ext"])
_simple_table_roundtrip(table)
+
+
+def test_undefined_logical_type(parquet_test_datadir):
+ test_file = f"{parquet_test_datadir}/data/unknown-logical-type.parquet"
+ if not os.path.exists(test_file):
+ pytest.skip("submodule not yet updated")
Review Comment:
I wasn't able to get passing tests on CI with this and I am not sure why. I
can remove the tests and follow up with a PR adding them, or I can follow up
with a PR removing this skip. (The C++ tests seem to pick up the updated
submodule that's included in this PR and the Python tests pass locally for me
without a skip).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
