[GitHub] [arrow-rs] alamb commented on a diff in pull request #1558: Add option to skip decoding arrow metadata from parquet (#1459)

[GitBox <gi...@apache.org>]

alamb commented on code in PR #1558:
URL: https://github.com/apache/arrow-rs/pull/1558#discussion_r850341604


##########
parquet/src/arrow/arrow_reader.rs:
##########
@@ -78,19 +78,40 @@ pub trait ArrowReader {
         T: IntoIterator<Item = usize>;
 }
 
+#[derive(Debug, Clone, Default)]
+pub struct ArrowReaderOptions {
+    skip_arrow_metadata: bool,
+}
+
+impl ArrowReaderOptions {
+    /// Create a new [`ArrowReaderOptions`] with the default settings
+    fn new() -> Self {
+        Self::default()
+    }
+
+    /// Parquet files generated by some writers may contain embedded arrow
+    /// schema and metadata. This may not be correct or compatible with your system
+    ///

Review Comment:
   ```suggestion
       /// schema and metadata. This may not be correct or compatible with your system.
       /// 
       /// For example:[ARROW-16184](https://issues.apache.org/jira/browse/ARROW-16184)
       ///
   
   ```



-- 
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-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org