fix: resolve all signed/unsigned integer comparison warnings

- Fix PARSE_PRIMITIVE_FIELD macro (line 42-43)
- Fix PARSE_STRING_FIELD macro (line 54-55)
- Fix PARSE_BINARY_FIELD macro (line 66-67)
- Fix view_of_file_field->length loop (line 360-361)
- Fix view_of_partition->length loop (line 376-377)
- Use static_cast<size_t>() for all Arrow array length comparisons
- Resolves all modernize-use-integer-sign-comparison clang-tidy warnings

Author: HeartLinked

src/iceberg/manifest_reader_internal.cc

@@ -39,7 +39,8 @@ namespace iceberg {
   }
 
 #define PARSE_PRIMITIVE_FIELD(item, array_view, type)                                   \
-  for (size_t row_idx = 0; row_idx < array_view->length; row_idx++) {                   \
+  for (size_t row_idx = 0; row_idx < static_cast<size_t>(array_view->length);           \
+       row_idx++) {                                                                     \
     if (!ArrowArrayViewIsNull(array_view, row_idx)) {                                   \
       auto value = ArrowArrayViewGetIntUnsafe(array_view, row_idx);                     \
       item = static_cast<type>(value);                                                  \
@@ -50,7 +51,8 @@ namespace iceberg {
   }
 
 #define PARSE_STRING_FIELD(item, array_view)                                            \
-  for (size_t row_idx = 0; row_idx < array_view->length; row_idx++) {                   \
+  for (size_t row_idx = 0; row_idx < static_cast<size_t>(array_view->length);           \
+       row_idx++) {                                                                     \
     if (!ArrowArrayViewIsNull(array_view, row_idx)) {                                   \
       auto value = ArrowArrayViewGetStringUnsafe(array_view, row_idx);                  \
       item = std::string(value.data, value.size_bytes);                                 \
@@ -61,7 +63,8 @@ namespace iceberg {
   }
 
 #define PARSE_BINARY_FIELD(item, array_view)                                            \
-  for (size_t row_idx = 0; row_idx < array_view->length; row_idx++) {                   \
+  for (size_t row_idx = 0; row_idx < static_cast<size_t>(array_view->length);           \
+       row_idx++) {                                                                     \
     if (!ArrowArrayViewIsNull(view_of_column, row_idx)) {                               \
       item = ArrowArrayViewGetInt8Vector(array_view, row_idx);                          \
     } else if (required) {                                                              \
@@ -357,7 +360,8 @@ Status ParseDataFile(const std::shared_ptr<StructType>& data_file_schema,
                            view_of_file_field);
         break;
       case 2:
-        for (size_t row_idx = 0; row_idx < view_of_file_field->length; row_idx++) {
+        for (size_t row_idx = 0;
+             row_idx < static_cast<size_t>(view_of_file_field->length); row_idx++) {
           if (!ArrowArrayViewIsNull(view_of_file_field, row_idx)) {
             auto value = ArrowArrayViewGetStringUnsafe(view_of_file_field, row_idx);
             std::string_view path_str(value.data, value.size_bytes);