zhuqi-lucas commented on code in PR #15239:
URL: https://github.com/apache/datafusion/pull/15239#discussion_r2000189083
##########
datafusion/common/src/dfschema.rs:
##########
@@ -605,25 +582,49 @@ impl DFSchema {
}
/// Returns true if the two schemas have the same qualified named
- /// fields with the same data types. Returns false otherwise.
+ /// fields with the compatible data types. Returns false otherwise.
///
/// This is a specialized version of Eq that ignores differences
/// in nullability and metadata.
///
/// Use [DFSchema]::logically_equivalent_names_and_types for a weaker
/// logical type checking, which for example would consider a dictionary
/// encoded UTF8 array to be equivalent to a plain UTF8 array.
- pub fn equivalent_names_and_types(&self, other: &Self) -> bool {
+ pub fn equivalent_names_and_types(&self, other: &Self) -> Result<()> {
Review Comment:
Thank you @alamb for view, got it, good guide for me to upgrade. Addressed
in latest PR.
--
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
