fix(blob): address review comments on include hygiene and test isolation

Include jdo_error.h directly in jindo_utils.h instead of relying on the
transitive include from JdoStatus.hpp, and give each writer in
BlobWriterBuilderTest.TestWriteNullOptions its own output file so that
a Finish() in one scenario cannot leak bytes into the next.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

Author: SteNicholas

src/paimon/format/blob/blob_writer_builder_test.cpp

@@ -65,11 +65,18 @@ TEST_F(BlobWriterBuilderTest, TestWriteNullOptions) {
         PAIMON_RETURN_NOT_OK_FROM_ARROW(arrow::ExportArray(*array, c_array));
         return Status::OK();
     };
+    // Each writer gets its own output file so that a Finish() in one scenario cannot leak
+    // bytes into the next.
+    int32_t file_index = 0;
     auto build_writer = [&](const std::map<std::string, std::string>& options)
         -> Result<std::unique_ptr<FormatWriter>> {
+        PAIMON_ASSIGN_OR_RAISE(
+            std::shared_ptr<OutputStream> out,
+            file_system_->Create(dir_->Str() + "/file" + std::to_string(file_index++) + ".blob",
+                                 /*overwrite=*/true));
         BlobWriterBuilder builder(struct_type_, options);
         builder.WithFileSystem(file_system_);
-        return builder.Build(output_stream_, "none");
+        return builder.Build(out, "none");
     };
 
     ASSERT_OK_AND_ASSIGN(std::shared_ptr<Blob> missing_blob,

src/paimon/fs/jindo/jindo_utils.h

@@ -17,6 +17,7 @@
 #pragma once
 
 #include "JdoStatus.hpp"  // NOLINT(build/include_subdir)
+#include "jdo_error.h"    // NOLINT(build/include_subdir)
 #include "paimon/status.h"
 
 namespace paimon::jindo {