blaginin commented on code in PR #15165:
URL: https://github.com/apache/datafusion/pull/15165#discussion_r1990156623
##########
datafusion/common/src/test_util.rs:
##########
@@ -73,6 +76,31 @@ macro_rules! assert_batches_eq {
};
}
+pub fn batches_to_string(batches: &[RecordBatch]) -> String {
+ let actual = pretty_format_batches_with_options(batches,
&DEFAULT_FORMAT_OPTIONS)
+ .unwrap()
+ .to_string();
+
+ actual.trim().to_string()
+}
+
+pub fn batches_to_sort_string(batches: &[RecordBatch]) -> String {
Review Comment:
there's alternative way: use
https://docs.rs/insta/latest/insta/struct.Settings.html#method.sort_maps
this, however, will require switching to serializable format - which we dont
want - so leaving as is
--
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...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
