klion26 commented on code in PR #7935:
URL: https://github.com/apache/arrow-rs/pull/7935#discussion_r2209376279


##########
parquet-variant/src/builder.rs:
##########
@@ -955,13 +967,48 @@ impl<'a> ObjectBuilder<'a> {
 
     // Returns validate_unique_fields because we can no longer reference self 
once this method returns.
     fn parent_state<'b>(&'b mut self, key: &'b str) -> (ParentState<'b>, bool) 
{
-        let state = ParentState::Object {
-            buffer: &mut self.buffer,
-            metadata_builder: self.parent_state.metadata_builder(),
-            fields: &mut self.fields,
-            field_name: key,
-        };
-        (state, self.validate_unique_fields)
+        let validate_unique_fields = self.validate_unique_fields;
+
+        match &mut self.parent_state {

Review Comment:
   Did find a better solution for this, I can change this if there is a better 
solution.



-- 
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...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to