refactor(predicate): move predicate_utils.h to public include (#181)

Move predicate_utils.h from src/paimon/common/predicate/ to
include/paimon/predicate/ to expose it as a public API header.

Changes:
- Relocate predicate_utils.h to include/paimon/predicate/
- Update all include paths across the codebase
- Remove fmt/format.h dependency from header
- Replace fmt::format with string concatenation in error msg

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: mrdrivingduck

…aimon/common/predicate/predicate_utils.h …clude/paimon/predicate/predicate_utils.hsrc/paimon/common/predicate/predicate_utils.h renamed to include/paimon/predicate/predicate_utils.h src/paimon/common/predicate/predicate_utils.h renamed to include/paimon/predicate/predicate_utils.h

@@ -24,7 +24,6 @@
 #include <string>
 #include <vector>
 
-#include "fmt/format.h"
 #include "paimon/predicate/function.h"
 #include "paimon/predicate/function_visitor.h"
 #include "paimon/predicate/leaf_predicate.h"
@@ -114,8 +113,8 @@ class PAIMON_EXPORT PredicateUtils {
                 return visitor->VisitLike(predicate->Literals()[0]);
             }
             default:
-                return Status::Invalid(fmt::format("invalid {} function in leaf predicate",
-                                                   predicate->GetFunction().ToString()));
+                return Status::Invalid("invalid " + predicate->GetFunction().ToString() +
+                                       " function in leaf predicate");
         }
     }
 

src/paimon/common/predicate/predicate_utils.cpp

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-#include "paimon/common/predicate/predicate_utils.h"
+#include "paimon/predicate/predicate_utils.h"
 
 #include <utility>
 

src/paimon/common/predicate/predicate_utils_test.cpp

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-#include "paimon/common/predicate/predicate_utils.h"
+#include "paimon/predicate/predicate_utils.h"
 
 #include <utility>
 

src/paimon/core/global_index/global_index_evaluator_impl.cpp

@@ -17,9 +17,9 @@
 #include "paimon/core/global_index/global_index_evaluator_impl.h"
 
 #include "fmt/format.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/global_index/bitmap_global_index_result.h"
 #include "paimon/predicate/leaf_predicate.h"
+#include "paimon/predicate/predicate_utils.h"
 
 namespace paimon {
 Result<std::optional<std::shared_ptr<GlobalIndexResult>>> GlobalIndexEvaluatorImpl::Evaluate(

src/paimon/core/io/file_index_evaluator.cpp

@@ -26,7 +26,6 @@
 #include "arrow/type.h"
 #include "fmt/format.h"
 #include "fmt/ranges.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/utils/arrow/status_utils.h"
 #include "paimon/common/utils/date_time_utils.h"
 #include "paimon/common/utils/field_type_utils.h"
@@ -42,6 +41,7 @@
 #include "paimon/predicate/leaf_predicate.h"
 #include "paimon/predicate/literal.h"
 #include "paimon/predicate/predicate.h"
+#include "paimon/predicate/predicate_utils.h"
 #include "paimon/status.h"
 
 namespace paimon {

src/paimon/core/operation/append_only_file_store_scan.cpp

@@ -28,7 +28,6 @@
 #include "arrow/type.h"
 #include "fmt/format.h"
 #include "paimon/common/predicate/predicate_filter.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/types/data_field.h"
 #include "paimon/core/core_options.h"
 #include "paimon/core/io/data_file_meta.h"
@@ -40,6 +39,7 @@
 #include "paimon/core/stats/simple_stats_evolutions.h"
 #include "paimon/core/utils/field_mapping.h"
 #include "paimon/file_index/file_index_result.h"
+#include "paimon/predicate/predicate_utils.h"
 #include "paimon/status.h"
 
 namespace paimon {

src/paimon/core/operation/file_store_scan.cpp

@@ -29,7 +29,6 @@
 #include "paimon/common/data/binary_array.h"
 #include "paimon/common/executor/future.h"
 #include "paimon/common/predicate/literal_converter.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/types/data_field.h"
 #include "paimon/common/utils/field_type_utils.h"
 #include "paimon/core/io/data_file_meta.h"
@@ -48,6 +47,7 @@
 #include "paimon/predicate/literal.h"
 #include "paimon/predicate/predicate.h"
 #include "paimon/predicate/predicate_builder.h"
+#include "paimon/predicate/predicate_utils.h"
 #include "paimon/scan_context.h"
 
 namespace paimon {

src/paimon/core/operation/key_value_file_store_scan.cpp

@@ -27,7 +27,6 @@
 #include "paimon/common/data/binary_array.h"
 #include "paimon/common/data/binary_row.h"
 #include "paimon/common/predicate/predicate_filter.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/utils/object_utils.h"
 #include "paimon/core/core_options.h"
 #include "paimon/core/io/data_file_meta.h"
@@ -37,6 +36,7 @@
 #include "paimon/core/stats/simple_stats_evolution.h"
 #include "paimon/core/stats/simple_stats_evolutions.h"
 #include "paimon/predicate/predicate.h"
+#include "paimon/predicate/predicate_utils.h"
 
 namespace arrow {
 class Schema;

src/paimon/core/operation/merge_file_split_read.cpp

@@ -27,7 +27,6 @@
 #include "arrow/c/abi.h"
 #include "arrow/c/bridge.h"
 #include "arrow/type.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/reader/complete_row_kind_batch_reader.h"
 #include "paimon/common/reader/concat_batch_reader.h"
 #include "paimon/common/table/special_fields.h"
@@ -64,6 +63,7 @@
 #include "paimon/core/utils/file_store_path_factory.h"
 #include "paimon/core/utils/primary_key_table_utils.h"
 #include "paimon/memory/memory_pool.h"
+#include "paimon/predicate/predicate_utils.h"
 #include "paimon/reader/file_batch_reader.h"
 #include "paimon/table/source/data_split.h"
 #include "paimon/utils/roaring_bitmap32.h"

src/paimon/core/utils/field_mapping.cpp

@@ -25,14 +25,14 @@
 #include "fmt/format.h"
 #include "paimon/common/predicate/compound_predicate_impl.h"
 #include "paimon/common/predicate/leaf_predicate_impl.h"
-#include "paimon/common/predicate/predicate_utils.h"
 #include "paimon/common/utils/field_type_utils.h"
 #include "paimon/common/utils/object_utils.h"
 #include "paimon/core/casting/cast_executor_factory.h"
 #include "paimon/core/casting/casting_utils.h"
 #include "paimon/defs.h"
 #include "paimon/predicate/literal.h"
 #include "paimon/predicate/predicate_builder.h"
+#include "paimon/predicate/predicate_utils.h"
 #include "paimon/status.h"
 
 namespace paimon {