andygrove commented on a change in pull request #7133:
URL: https://github.com/apache/arrow/pull/7133#discussion_r426705792



##########
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:
       I don't have a strong opinion in this either. If you found it useful for 
debugging then perhaps other users would too. @nevi-me What do you think? Any 
reason not to enable Debug here? Presumably there is no cost to this unless 
people choose to print them?




----------------------------------------------------------------
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]


Reply via email to