[jira] [Commented] (ARROW-16947) [C++] Remove boost dependency with thrift

["Kouhei Sutou (Jira)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/ARROW-16947?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17610298#comment-17610298 ] 

Kouhei Sutou commented on ARROW-16947:
--------------------------------------

FYI: https://github.com/apache/thrift/pull/2630 was merged. It means that we don't need Boost with system Thrift 0.17.0 or later.

We need more work to remove Boost dependency for bundled Thrift. Because we still need Boost for building Thrift. But Thrift developers want to remove build-time Boost dependency too: https://github.com/apache/thrift/pull/2630#issuecomment-1242712937

{quote}
Thanks to you! If you have more such changes, please keep them coming!
{quote}

So, our contributions to remove build-time Boost dependency will be welcome.

> [C++] Remove boost dependency with thrift
> -----------------------------------------
>
>                 Key: ARROW-16947
>                 URL: https://issues.apache.org/jira/browse/ARROW-16947
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Neal Richardson
>            Assignee: Kouhei Sutou
>            Priority: Major
>              Labels: good-second-issue
>
> [~kou] (re-)added this dependency in ARROW-16721: https://github.com/apache/arrow/pull/13292/files#r890849903. But looking at thrift/transport/TBufferTransports.h, the header we include that uses boost, the class we use from it doesn't seem to require boost itself. So maybe we can pull the class definition out that we need and inline/vendor it, so that we can drop the need for that header, and thus drop the need for boost with thrift.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)