rok commented on code in PR #7283: URL: https://github.com/apache/arrow-rs/pull/7283#discussion_r1994484025
########## parquet/src/arrow/mod.rs: ########## @@ -93,6 +93,96 @@ //! //! println!("Read {} records.", record_batch.num_rows()); //! ``` +//! +//! # Example of reading uniformly encrypted parquet file into arrow record batch +//! +#![cfg_attr(feature = "encryption", doc = "```rust")] +#![cfg_attr(not(feature = "encryption"), doc = "```ignore")] +//! # use arrow_array::{Int32Array, ArrayRef}; +//! # use arrow_array::{types, RecordBatch}; Review Comment: Ha, it seems that sometimes docs are [tested without encryption](https://github.com/apache/arrow-rs/actions/runs/13847026024/job/38747477104?pr=7283) so this would fail. I'm reintroducing the `cfg` check for now. However it's not clear to me if the docs that will ultimately be produced will include this example or not. -- 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: github-unsubscr...@arrow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org