fix: fix pre-commit error

Author: lucasfang

cmake_modules/BuildUtils.cmake

@@ -291,9 +291,7 @@ function(add_test_case REL_TEST_NAME)
         add_dependencies(${LABEL_TEST_NAME} ${TEST_NAME})
     endforeach()
 
-    set_property(TEST ${TEST_NAME}
-                 APPEND
-                 PROPERTY LABELS ${LABELS})
+    set_property(TEST ${TEST_NAME} APPEND PROPERTY LABELS ${LABELS})
 endfunction()
 
 # Adding unit tests part of the "paimon" portion of the test suite

cmake_modules/ThirdpartyToolchain.cmake

@@ -251,7 +251,8 @@ macro(build_fmt)
     externalproject_add(fmt_ep
                         URL ${FMT_SOURCE_URL}
                         URL_HASH "SHA256=${PAIMON_FMT_BUILD_SHA256_CHECKSUM}"
-                        ${FMT_CONFIGURE} ${THIRDPARTY_LOG_OPTIONS}
+                        ${FMT_CONFIGURE}
+                        ${THIRDPARTY_LOG_OPTIONS}
                         BUILD_BYPRODUCTS ${FMT_STATIC_LIB})
 
     file(MAKE_DIRECTORY "${FMT_INCLUDE_DIR}")
@@ -339,7 +340,8 @@ macro(build_zstd)
     externalproject_add(zstd_ep
                         URL ${ZSTD_SOURCE_URL}
                         URL_HASH "SHA256=${PAIMON_ZSTD_BUILD_SHA256_CHECKSUM}"
-                        ${ZSTD_CONFIGURE} ${THIRDPARTY_LOG_OPTIONS}
+                        ${ZSTD_CONFIGURE}
+                        ${THIRDPARTY_LOG_OPTIONS}
                         BUILD_BYPRODUCTS ${ZSTD_STATIC_LIB})
 
     file(MAKE_DIRECTORY "${ZSTD_INCLUDE_DIR}")
@@ -365,7 +367,8 @@ macro(build_lz4)
     externalproject_add(lz4_ep
                         URL ${LZ4_SOURCE_URL}
                         URL_HASH "SHA256=${PAIMON_LZ4_BUILD_SHA256_CHECKSUM}"
-                        ${LZ4_CONFIGURE} ${THIRDPARTY_LOG_OPTIONS}
+                        ${LZ4_CONFIGURE}
+                        ${THIRDPARTY_LOG_OPTIONS}
                         BUILD_BYPRODUCTS ${LZ4_STATIC_LIB})
 
     file(MAKE_DIRECTORY "${LZ4_INCLUDE_DIR}")
@@ -507,7 +510,8 @@ macro(build_protobuf)
     externalproject_add(protobuf_ep
                         URL ${PROTOBUF_SOURCE_URL}
                         URL_HASH "SHA256=${PAIMON_PROTOBUF_BUILD_SHA256_CHECKSUM}"
-                        ${PROTOBUF_CONFIGURE} ${THIRDPARTY_LOG_OPTIONS}
+                        ${PROTOBUF_CONFIGURE}
+                        ${THIRDPARTY_LOG_OPTIONS}
                         # BUILD_IN_SOURCE 1
                         BUILD_BYPRODUCTS "${PROTOBUF_STATIC_LIB}" "${PROTOBUF_COMPILER}"
                         DEPENDS zlib)

docs/source/implementations/api.rst

@@ -33,5 +33,5 @@ API Reference
    api/global_index
    api/clean
    api/defs
-   api/executor 
-   api/memory 
+   api/executor
+   api/memory

docs/source/implementations/api/executor.rst

@@ -31,4 +31,4 @@ Interface
 
 .. doxygenclass:: paimon::Executor
    :members:
-   :undoc-members:
+   :undoc-members:

docs/source/implementations/user_guide/read.rst

@@ -81,7 +81,7 @@ return values between implementations.
 Type Change Support Matrix
 --------------------------
 
-The table below indicates support for changing a column type from `source` to `target`. Refer to the numbered notes below the table
+The table below indicates support for changing a column type from ``source`` to ``target``. Refer to the numbered notes below the table
 for caveats.
 
 .. list-table::

examples/CMakeLists.txt

@@ -38,4 +38,4 @@ target_link_libraries(clean_demo
                               paimon_shared
                               paimon_parquet_file_format_shared
                               paimon_orc_file_format_shared
-                              paimon_local_file_system_shared)
+                              paimon_local_file_system_shared)

src/paimon/CMakeLists.txt

@@ -46,7 +46,7 @@ set(PAIMON_COMMON_SRCS
     common/fs/file_system.cpp
     common/fs/resolving_file_system.cpp
     common/fs/file_system_factory.cpp
-    common/global_index/bitmap_topk_global_index_result.cpp    
+    common/global_index/bitmap_topk_global_index_result.cpp
     common/global_index/bitmap_global_index_result.cpp
     common/global_index/global_index_result.cpp
     common/global_index/global_indexer_factory.cpp
@@ -147,7 +147,7 @@ set(PAIMON_CORE_SRCS
     core/global_index/row_range_global_index_writer.cpp
     core/index/index_file_handler.cpp
     core/index/global_index_meta.cpp
-    core/index/index_file_meta_serializer.cpp    
+    core/index/index_file_meta_serializer.cpp
     core/io/meta_to_arrow_array_converter.cpp
     core/io/async_key_value_producer_and_consumer.cpp
     core/io/data_file_meta_09_serializer.cpp
@@ -180,7 +180,7 @@ set(PAIMON_CORE_SRCS
     core/manifest/manifest_file_meta_serializer.cpp
     core/manifest/manifest_list.cpp
     core/manifest/partition_entry.cpp
-    core/manifest/index_manifest_file_handler.cpp    
+    core/manifest/index_manifest_file_handler.cpp
     core/mergetree/compact/aggregate/aggregate_merge_function.cpp
     core/mergetree/compact/aggregate/field_sum_agg.cpp
     core/mergetree/compact/interval_partition.cpp

src/paimon/core/operation/data_evolution_file_store_scan.h

@@ -91,7 +91,7 @@ class DataEvolutionFileStoreScan : public FileStoreScan {
     static bool FilterManifestByRowRanges(const ManifestFileMeta& manifest,
                                           const std::vector<Range>& row_ranges);
     static Result<bool> FilterEntryByRowRanges(const ManifestEntry& entry,
-                                       const std::vector<Range>& row_ranges);
+                                               const std::vector<Range>& row_ranges);
 
     static Result<std::pair<int64_t, SimpleStatsEvolution::EvolutionStats>> EvolutionStats(
         const std::vector<ManifestEntry>& entries, const std::shared_ptr<TableSchema>& table_schema,

src/paimon/core/operation/data_evolution_file_store_scan_test.cpp

@@ -601,7 +601,7 @@ TEST_F(DataEvolutionFileStoreScanTest, TestFilterManifestByRowRanges) {
         manifest1, {Range(0, 15), Range(100, 200)}));
     ASSERT_FALSE(DataEvolutionFileStoreScan::FilterManifestByRowRanges(
         manifest1, {Range(0, 5), Range(100, 200)}));
-    
+
     auto manifest2 =
         ManifestFileMeta("manifest-65b0d403-a1bc-4157-b242-bff73c46596d-0", /*file_size=*/2779,
                          /*num_added_files=*/1, /*num_deleted_files=*/0, SimpleStats::EmptyStats(),

src/paimon/core/operation/read_context.cpp

@@ -209,11 +209,12 @@ Result<std::unique_ptr<ReadContext>> ReadContextBuilder::Finish() {
         return Status::Invalid("row to batch thread number should be greater than 0");
     }
     auto ctx = std::make_unique<ReadContext>(
-        impl_->path_, impl_->branch_, impl_->read_field_names_, impl_->predicate_, impl_->row_ranges_,
-        impl_->enable_predicate_filter_, impl_->enable_prefetch_, impl_->prefetch_batch_count_,
-        impl_->prefetch_max_parallel_num_, impl_->enable_multi_thread_row_to_batch_,
-        impl_->row_to_batch_thread_number_, impl_->table_schema_, impl_->memory_pool_,
-        impl_->executor_, impl_->fs_scheme_to_identifier_map_, impl_->options_);
+        impl_->path_, impl_->branch_, impl_->read_field_names_, impl_->predicate_,
+        impl_->row_ranges_, impl_->enable_predicate_filter_, impl_->enable_prefetch_,
+        impl_->prefetch_batch_count_, impl_->prefetch_max_parallel_num_,
+        impl_->enable_multi_thread_row_to_batch_, impl_->row_to_batch_thread_number_,
+        impl_->table_schema_, impl_->memory_pool_, impl_->executor_,
+        impl_->fs_scheme_to_identifier_map_, impl_->options_);
     impl_->Reset();
     return ctx;
 }