[
https://issues.apache.org/jira/browse/PARQUET-1233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16370411#comment-16370411
]
ASF GitHub Bot commented on PARQUET-1233:
-----------------------------------------
majetideepak commented on a change in pull request #443: PARQUET-1233: Enable
option to switch between stl classes and boost c…
URL: https://github.com/apache/parquet-cpp/pull/443#discussion_r169413745
##########
File path: CMakeLists.txt
##########
@@ -399,6 +399,11 @@ else()
endif()
include(ThirdpartyToolchain)
+if (THRIFT_VERSION VERSION_LESS "0.11.0")
+ add_definitions(-DPARQUET_THRIFT_USE_BOOST)
+ message(STATUS "Using Boost in Thrift header")
+endif()
Review comment:
Makes sense to add this configuration. Will include that.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> [CPP ]Enable option to switch between stl classes and boost classes for
> thrift header
> -------------------------------------------------------------------------------------
>
> Key: PARQUET-1233
> URL: https://issues.apache.org/jira/browse/PARQUET-1233
> Project: Parquet
> Issue Type: Bug
> Reporter: Deepak Majeti
> Assignee: Deepak Majeti
> Priority: Major
> Fix For: cpp-1.4.0
>
>
> Thrift 0.11.0 introduced breaking changes by defaulting to stl classes. This
> causes an issue with older thrift versions. The scope of this Jira is to
> enable an option to choose between stl and boost in parquet thrift header.
> https://thrift.apache.org/lib/cpp
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
