GH-39715: Adjust Field ordering in parquet::FileSerializer

apache · Jan 20, 2024 · c4933a9 · c4933a9
1 parent 55afcf0
commit c4933a9
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/cpp/src/parquet/file_writer.cc b/cpp/src/parquet/file_writer.cc
@@ -486,10 +486,12 @@ class FileSerializer : public ParquetFileWriter::Contents {
   int num_row_groups_;
   int64_t num_rows_;
   std::unique_ptr<FileMetaDataBuilder> metadata_;
+  // GH-39715: file_encryptor_ must be destroyed after page_index_builder_,
+  // and page_index_builder_ must be destroyed after row_group_writer_.
+  std::unique_ptr<InternalFileEncryptor> file_encryptor_;
+  std::unique_ptr<PageIndexBuilder> page_index_builder_;
   // Only one of the row group writers is active at a time
   std::unique_ptr<RowGroupWriter> row_group_writer_;
-  std::unique_ptr<PageIndexBuilder> page_index_builder_;
-  std::unique_ptr<InternalFileEncryptor> file_encryptor_;
 
   void StartFile() {
     auto file_encryption_properties = properties_->file_encryption_properties();