Jefffrey commented on code in PR #11167:
URL: https://github.com/apache/arrow-rs/pull/11167#discussion_r4068048786
##########
.github/workflows/parquet.yml:
##########
@@ -43,6 +43,8 @@ on:
- arrow-json/**
- arrow-avro/**
- parquet/**
+ - parquet_derive/**
+ - parquet_derive_test/**
Review Comment:
we dont get too many frequent changes to derive crates anyway (most recent
ones are just wider workspace fixes)
##########
.github/workflows/parquet.yml:
##########
@@ -73,6 +75,10 @@ jobs:
# Test parquet examples
cargo run -p parquet --example read_parquet
cargo run -p parquet --example read_with_rowgroup --features="async"
+ - name: Test (parquet_derive)
+ run: |
+ cargo test -p parquet_derive
+ cargo test -p parquet_derive_test
Review Comment:
its actually a fix to include running `cargo test -p parquet_derive_test`;
seems we never did apart from running it on mac/windows runners (since they run
whole workspace test suite)
--
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]
