fix ci

Author: HuaHuaY

cpp/src/arrow/extension/variant/meson.build

@@ -17,19 +17,9 @@
 
 exc = executable(
     'arrow-variant-extension-test',
-    sources: [
-        'type_test.cc',
-        'encoding_test.cc',
-        'builder_test.cc',
-    ],
+    sources: ['type_test.cc', 'encoding_test.cc', 'builder_test.cc'],
     dependencies: [arrow_test_dep],
 )
 test('arrow-variant-extension-test', exc)
 
-install_headers(
-    [
-        'encoding.h',
-        'builder.h',
-    ],
-    subdir: 'arrow/extension/variant',
-)
+install_headers(['encoding.h', 'builder.h'], subdir: 'arrow/extension/variant')

cpp/src/arrow/extension/variant/type_test.cc

@@ -26,9 +26,9 @@ TEST(TestVariantType, Storage) {
   auto unshredded = struct_({field("metadata", binary(), /*nullable=*/false),
                              field("value", binary(), /*nullable=*/false)});
   ASSERT_OK_AND_ASSIGN(auto type, VariantExtensionType::Make(unshredded));
-  ASSERT_EQ(VariantExtensionType::kExtensionName,
-            ::arrow::internal::checked_cast<const ExtensionType&>(*type)
-                .extension_name());
+  ASSERT_EQ(
+      VariantExtensionType::kExtensionName,
+      ::arrow::internal::checked_cast<const ExtensionType&>(*type).extension_name());
 
   auto shredded = struct_({field("metadata", binary(), /*nullable=*/false),
                            field("value", binary()), field("typed_value", int64())});

cpp/src/parquet/arrow/arrow_reader_writer_test.cc

@@ -5928,11 +5928,8 @@ TEST(TestArrowReadWrite, AllNulls) {
   auto schema = ::arrow::schema({::arrow::field("all_nulls", ::arrow::int8())});
 
   constexpr int64_t length = 3;
-  ASSERT_OK_AND_ASSIGN(auto null_bitmap, ::arrow::AllocateEmptyBitmap(length));
-  auto array_data = ::arrow::ArrayData::Make(
-      ::arrow::int8(), length, {null_bitmap, /*values=*/nullptr}, /*null_count=*/length);
-  auto array = ::arrow::MakeArray(array_data);
-  auto record_batch = ::arrow::RecordBatch::Make(schema, length, {array});
+  ASSERT_OK_AND_ASSIGN(auto array, MakeArrayOfNull(::arrow::int8(), length));
+  auto record_batch = ::arrow::RecordBatch::Make(schema, length, {std::move(array)});
 
   auto sink = CreateOutputStream();
   ASSERT_OK_AND_ASSIGN(auto writer, parquet::arrow::FileWriter::Open(

cpp/src/parquet/arrow/variant_validate_internal.h

@@ -18,6 +18,7 @@
 #pragma once
 
 #include "arrow/status.h"
+#include "parquet/platform.h"
 
 namespace arrow {
 
@@ -28,6 +29,7 @@ class MemoryPool;
 
 namespace parquet::arrow::internal {
 
+PARQUET_EXPORT
 ::arrow::Status ValidateVariants(const ::arrow::ChunkedArray& data,
                                  ::arrow::MemoryPool* pool);