Merge pull request #63 from ClickHouse/try-fix-abort

Don't throw in ~DictEncoderImpl()
ClickHouse · Apr 3, 2024 · 8f36d71 · 8f36d71
2 parents ba5c679 + 46e7ed1
commit 8f36d71
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/cpp/src/parquet/encoding.cc b/cpp/src/parquet/encoding.cc
@@ -499,7 +499,11 @@ class DictEncoderImpl : public EncoderImpl, virtual public DictEncoder<DType> {
         dict_encoded_size_(0),
         memo_table_(pool, kInitialHashTableSize) {}
 
-  ~DictEncoderImpl() override { DCHECK(buffered_indices_.empty()); }
+  ~DictEncoderImpl() override {
+    if (!buffered_indices_.empty()) {
+      ClearIndices();
+    }
+  }
 
   int dict_encoded_size() const override { return dict_encoded_size_; }