refactor: use template function for manifest reader

[wgtmac]

• Removed PARSE_XXX macros as many as possible
• Refined error reporting

[HuaHuaY]

I suggest to use a template to merge ParseIntegerField, ParseStringField and ParseBinaryField and simplify Accessor to just receive a projection.

template <typename Container, typename Accessor, typename Transfer>
  requires std::ranges::forward_range<Container> &&
           requires(Accessor& a, Container& c, Transfer& t, const ArrowArrayView* v,
                    int64_t i) {
             std::invoke(a, *std::ranges::begin(c)) = std::invoke(t, v, i);
           }
Status ParseField(const ArrowArrayView* array_view, Container& container,
                  Accessor accessor, std::string_view field_name, bool required,
                  ErrorKind error_kind, Transfer transfer) {
  auto iter = std::ranges::begin(container);
  for (int64_t row_idx = 0; row_idx < array_view->length; row_idx++, iter++) {
    if (!ArrowArrayViewIsNull(array_view, row_idx)) {
      std::invoke(accessor, *iter) = std::invoke(transfer, array_view, row_idx);
    } else if (required) {
      return MakeNullError(error_kind, field_name, row_idx);
    }
  }
  return {};
}

template <typename T, typename Container, typename Accessor>
Status ParseIntegerField(const ArrowArrayView* array_view, Container& container,
                         Accessor accessor, std::string_view field_name, bool required,
                         ErrorKind error_kind) {
  return ParseField(array_view, container, accessor, field_name, required, error_kind,
                    [](const ArrowArrayView* view, int64_t row_idx) {
                      return static_cast<T>(ArrowArrayViewGetIntUnsafe(view, row_idx));
                    });
}

template <typename Container, typename Accessor>
Status ParseStringField(const ArrowArrayView* array_view, Container& container,
                        Accessor accessor, std::string_view field_name, bool required,
                        ErrorKind error_kind) {
  return ParseField(array_view, container, accessor, field_name, required, error_kind,
                    [](const ArrowArrayView* view, int64_t row_idx) {
                      auto value = ArrowArrayViewGetStringUnsafe(view, row_idx);
                      return std::string(value.data, value.size_bytes);
                    });
}

template <typename Container, typename Accessor>
Status ParseBinaryField(const ArrowArrayView* array_view, Container& container,
                        Accessor accessor, std::string_view field_name, bool required,
                        ErrorKind error_kind) {
  return ParseField(array_view, container, accessor, field_name, required, error_kind,
                    ParseInt8VectorField);
}

[HuaHuaY]

After offline discussion, I will open a new PR to modify this.