friendlymatthew commented on PR #7741: URL: https://github.com/apache/arrow-rs/pull/7741#issuecomment-3001796116
> So what i am saying is I think it would be correct for the VariantBuilder to just update the key and leave the old value there (but not referenced) 🤔 That would result in a larger final variant, but I think as long as we documented this behavior it would be ok from the user perspective (I am envisioning many different possible desired optimizations for variant creation) I wonder if we can add a `gc()` method to `VariantObject`. Similar to a `StringView`, users can call `VariantObject::gc` to undergo this expensive operation of pruning stale references, etc... -- 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
