[
https://issues.apache.org/jira/browse/ARROW-11735?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Neal Richardson updated ARROW-11735:
------------------------------------
Description:
Parquet requires thrift and it seems that thrift (at least as of version 0.12)
does not compile on Solaris:
{code}
/export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:
In member function virtual void
apache::thrift::transport::THttpServer::parseHeader(char*):
/export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:50:74:
error: strcasestr was not declared in this scope
#define THRIFT_strcasestr(haystack, needle) strcasestr(haystack, needle)
^
/export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:62:9:
note: in expansion of macro THRIFT_strcasestr
if (THRIFT_strcasestr(value, "chunked") != NULL) {
{code}
(along with some boost endian header deprecation warnings)
We could debug/patch that, or we could also make Parquet an optional feature in
the R bindings. That might have some value anyway so that one could build a
lighter/minimal R package, if that were helpful.
was:Parquet requires thrift and it seems that thrift (at least as of version
0.12) does not compile on Solaris. We could debug that, or we could also make
Parquet an optional feature in the R bindings. That might have some value
anyway so that one could build a lighter/minimal R package, if that were
helpful.
> [R] Allow parquet to be an optional component like S3
> -----------------------------------------------------
>
> Key: ARROW-11735
> URL: https://issues.apache.org/jira/browse/ARROW-11735
> Project: Apache Arrow
> Issue Type: Sub-task
> Components: R
> Reporter: Neal Richardson
> Priority: Major
> Fix For: 4.0.0
>
>
> Parquet requires thrift and it seems that thrift (at least as of version
> 0.12) does not compile on Solaris:
> {code}
> /export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:
> In member function virtual void
> apache::thrift::transport::THttpServer::parseHeader(char*):
> /export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:50:74:
> error: strcasestr was not declared in this scope
> #define THRIFT_strcasestr(haystack, needle) strcasestr(haystack, needle)
> ^
> /export/home/X1svPYR/Rtemp/RtmptF1MlN/file75097d284891/thrift_ep-prefix/src/thrift_ep/lib/cpp/src/thrift/transport/THttpServer.cpp:62:9:
> note: in expansion of macro THRIFT_strcasestr
> if (THRIFT_strcasestr(value, "chunked") != NULL) {
> {code}
> (along with some boost endian header deprecation warnings)
> We could debug/patch that, or we could also make Parquet an optional feature
> in the R bindings. That might have some value anyway so that one could build
> a lighter/minimal R package, if that were helpful.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
