NoahKusaba commented on code in PR #3303:
URL: https://github.com/apache/iceberg-rust/pull/3303#discussion_r4150543668
##########
crates/iceberg/src/arrow/schema.rs:
##########
@@ -1190,8 +1190,8 @@ pub(crate) fn
primitive_type_to_arrow_type_with_ree(primitive_type: &PrimitiveTy
/// including nested struct, list, and map fields. This is useful for schema
comparison
/// where metadata differences should be ignored.
struct MetadataStripVisitor {
- /// Stack to track field information during traversal
- field_stack: Vec<Field>,
+ /// Name and nullability of each field being visited, innermost last
+ field_stack: Vec<(String, bool)>,
Review Comment:
Yes, if you check the lines 1228 that I deleted, the "DataType" parameter of
Field had a useless dummy variable that was later discarded in a line like
1235.
This is a additional refactor so that MetadataStripVisitor isn't carrying
useless information.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
