houqp commented on a change in pull request #7133:
URL: https://github.com/apache/arrow/pull/7133#discussion_r426196139
##########
File path: rust/arrow/src/record_batch.rs
##########
@@ -36,7 +36,7 @@ use crate::error::{ArrowError, Result};
/// serialization and computation functions, possibly incremental.
/// See also [CSV reader](crate::csv::Reader) and
/// [JSON reader](crate::json::Reader).
-#[derive(Clone)]
+#[derive(Clone, Debug)]
Review comment:
Here is what it looks like:
```
RecordBatch {
schema: Schema {
fields: [
Field {
name: "id",
data_type: Int64,
nullable: true,
dict_id: 0,
dict_is_ordered: false,
},
],
metadata: {},
},
columns: [
PrimitiveArray<Int64>
[
5,
],
],
}
```
Again, I am happy to push a fixup to delete the Debug trait if anyone
prefers that. I don't have a strong opinion on this, just thought it could be
handy for future developers.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
