alamb commented on code in PR #8392:
URL: https://github.com/apache/arrow-rs/pull/8392#discussion_r2364024836
##########
parquet-variant-compute/src/variant_get.rs:
##########
@@ -154,11 +154,11 @@ fn shredded_get_path(
// Peel away the prefix of path elements that traverses the shredded parts
of this variant
// column. Shredding will traverse the rest of the path on a per-row basis.
- let mut shredding_state = input.shredding_state();
+ let mut shredding_state = input.shredding_state().clone();
let mut accumulated_nulls = input.inner().nulls().cloned();
let mut path_index = 0;
for path_element in path {
- match follow_shredded_path_element(shredding_state, path_element,
cast_options)? {
+ match follow_shredded_path_element(&shredding_state, path_element,
cast_options)? {
Review Comment:
it is necessary as it gets returned from the function
--
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
