Vineyard fails when compiling with Arrow 15.0.0

[raulcd]

Describe your problem

I released Arrow 15.0.0 a couple of days ago. I am following up distribution of the package and it seems the Homebrew PR is blocked because Vineyard is not compatible with Arrow 15.0.0: Homebrew/homebrew-core#160553 (comment)

  In file included from /tmp/vineyard-20240122-87256-on1dbr/v6d-0.20.2/modules/basic/ds/arrow_shim/concatenate.cc:67:
  /opt/homebrew/include/arrow/visit_type_inline.h:54:34: error: no matching member function for call to 'Visit'
      ARROW_GENERATE_FOR_ALL_TYPES(TYPE_VISIT_INLINE);
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~~~~
  /opt/homebrew/include/arrow/visitor_generate.h:43:3: note: expanded from macro 'ARROW_GENERATE_FOR_ALL_TYPES'
    ACTION(StringView);                           \
    ^~~~~~~~~~~~~~~~~~
  /opt/homebrew/include/arrow/visit_type_inline.h:29:21: note: expanded from macro 'TYPE_VISIT_INLINE'
      return visitor->Visit(internal::checked_cast<const TYPE_CLASS##Type&>(type), \
             ~~~~~~~~~^~~~~
  /tmp/vineyard-20240122-87256-on1dbr/v6d-0.20.2/modules/basic/ds/arrow_shim/concatenate.cc:255:19: note: in instantiation of function template specialization 'arrow::VisitTypeInline<vineyard::arrow_shim::(anonymous namespace)::ConcatenateImpl>' requested here
      RETURN_NOT_OK(VisitTypeInline(*out_->type, this));
                    ^
  /tmp/vineyard-20240122-87256-on1dbr/v6d-0.20.2/modules/basic/ds/arrow_shim/concatenate.cc:260:10: note: candidate function not viable: no known conversion from 'const arrow::StringViewType' to 'const NullType' for 1st argument
    Status Visit(const NullType&) { return Status::OK(); }
           ^

I can't give much details apart from adding compatibility with Arrow 15.

[sighingnow]

Thanks for reporting. Working on it.

[sighingnow]

Hi @raulcd, I have just uploaded a new release to github: https://github.com/v6d-io/v6d/releases/tag/v0.20.3.

Please bump up the version of vineyard in homebrew. Thanks!

[raulcd]

Thank you very much for the prompt response @sighingnow !

[timsutton]

Thanks @sighingnow! ❤️