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



##########
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 found it useful to have this trait for debugging purpose. It is not 
used in the change I made in this PR. I will remove it if the policy is to 
avoid adding Debug trait unless necessary.




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