[jira] [Updated] (ARROW-4319) plasma/store.h pulls ins flatbuffer dependency

["Matthias Vallentin (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/ARROW-4319?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Matthias Vallentin updated ARROW-4319:
--------------------------------------
    Attachment: patch.diff

> plasma/store.h pulls ins flatbuffer dependency
> ----------------------------------------------
>
>                 Key: ARROW-4319
>                 URL: https://issues.apache.org/jira/browse/ARROW-4319
>             Project: Apache Arrow
>          Issue Type: Bug
>          Components: C++
>    Affects Versions: 0.12.0
>            Reporter: Matthias Vallentin
>            Priority: Minor
>              Labels: build
>         Attachments: patch.diff
>
>   Original Estimate: 1h
>  Remaining Estimate: 1h
>
> For our unit testing, we'd like to use the plasma store programmatically by including *plasma/store.h*. However, this header pulls in flatbuffers via *src/plasma/common_generated.h*. Is this a necessary include or would a forward declaration suffice?
> Installing just flatbuffers didn't solve the problem, though. It looks like a specific version is needed:
> {noformat}
>  In file included from /Users/mavam/code/src/arrow/cpp/src/plasma/store.h:30:
>  In file included from /Users/mavam/code/src/arrow/cpp/src/plasma/eviction_policy.h:27:
>  In file included from /Users/mavam/code/src/arrow/cpp/src/plasma/plasma.h:41:
>  /Users/mavam/code/src/arrow/cpp/src/plasma/common_generated.h:65:21: error: no matching member function for call to 'Verify'
>  verifier.Verify(object_id()) &&
>  ~~~~~~~~~^~~~~~
>  /usr/local/include/flatbuffers/flatbuffers.h:1896:29: note: candidate template ignored: couldn't infer template argument 'T'
>  template<typename T> bool Verify(size_t elem) const {
>  ^
>  /usr/local/include/flatbuffers/flatbuffers.h:1905:29: note: candidate function template not viable: requires 2 arguments, but 1 was provided
>  template<typename T> bool Verify(const uint8_t *base, voffset_t elem_off)
>  ^
>  /usr/local/include/flatbuffers/flatbuffers.h:1880:8: note: candidate function not viable: requires 2 arguments, but 1 was provided
>  bool Verify(size_t elem, size_t elem_len) const {
>  ^
>  /usr/local/include/flatbuffers/flatbuffers.h:1901:8: note: candidate function not viable: requires 3 arguments, but 1 was provided
>  bool Verify(const uint8_t *base, voffset_t elem_off, size_t elem_len) const {
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)