Revert "[BugFix] Fix arrow parquet exception handling (StarRocks#25621)"

This reverts commit f610fdf.

be/src/formats/parquet/file_writer.cpp

@@ -377,7 +377,11 @@ Status SyncFileWriter::_flush_row_group() {
             _chunk_writer->close();
         } catch (const ::parquet::ParquetStatusException& e) {
             _chunk_writer.reset();
+
+            // this is to avoid calling ParquetFileWriter.Close which incurs segfault
             _closed = true;
+            _writer.release();
+
             auto st = Status::IOError(fmt::format("{}: {}", "flush rowgroup error", e.what()));
             LOG(WARNING) << st;
             return st;

thirdparty/download-thirdparty.sh

@@ -477,19 +477,13 @@ fi
 echo "Finished patching $SERDES_SOURCE"
 cd -
 
-# patch arrows
+# patch arrows to use our built jemalloc
 if [[ -d $TP_SOURCE_DIR/$ARROW_SOURCE ]] ; then
     cd $TP_SOURCE_DIR/$ARROW_SOURCE
-    # use our built jemalloc
     if [ ! -f $PATCHED_MARK ] && [ $ARROW_SOURCE = "arrow-apache-arrow-5.0.0" ] ; then
         patch -p1 < $TP_PATCH_DIR/arrow-5.0.0-force-use-external-jemalloc.patch
         touch $PATCHED_MARK
     fi
-    # fix arrow parquet exception handling
-    if [ ! -f $PATCHED_MARK ] && [ $ARROW_SOURCE = "arrow-apache-arrow-5.0.0" ] ; then
-        patch -p1 < $TP_PATCH_DIR/arrow-5.0.0-fix-exception-handling.patch
-        touch $PATCHED_MARK
-    fi
     cd -
     echo "Finished patching $ARROW_SOURCE"
 fi