[C++] Investigate dynamic linking for libthift

[asfimport]

We're currently only linking statically against libthrift . Distributions would often prefer a dynamic linkage to libraries where possible.

Reporter: Uwe Korn / @xhochy
Assignee: Uwe Korn / @xhochy

_{Note: This issue was originally created as ARROW-4447. Please see the migration documentation for further details.}

[asfimport]

Kevin Gurney / @kevingurney:
It looks like changes to support building libthrift as a shared library on Windows were only recently introduced in apache/thrift@278528c through the use of CMAKE_WINDOWS_EXPORT_ALL_SYMBOLS (as opposed to explicit usage of __declspec(dllexport)).

Therefore, it seems that supporting dynamic linking would require a version bump for Arrow's libthrift dependency to the latest upstream Thrift code beyond stable release 0.12.0.

Refer to https://issues.apache.org/jira/browse/THRIFT-1834 for further discussion.

[asfimport]

Uwe Korn / @xhochy:
Especially https://issues.apache.org/jira/browse/THRIFT-1834?focusedCommentId=13557513&page=com.atlassian.jira.plugin.system.issuetabpanels%3Acomment-tabpanel#comment-13557513 advises against dynamic linking at the moment. We should reinvestigate this in the future.

[asfimport]

Wes McKinney / @wesm:
Thrift is now dynamically linked