GitHub user kostrzewa9ld created a discussion: Semantics of 
FileMetaDataBuilder::Finish() and saving memory by moving std::vector<RowGroup> 
metadata

I've noticed that in `FileMetaDataBuilderImpl::Finish()` there is a copy of 
`row_groups_` field done by passing it via `const&` to 
`metadata_->__set_row_groups(row_groups_)`. However, if it is assumed that 
after calling the `Finish()` method nothing should be touching the 
`row_groups_` field anymore then maybe it could be moved instead? 

GitHub link: https://github.com/apache/arrow/discussions/47676

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to