You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Neal Richardson (Jira)" <ji...@apache.org> on 2021/03/06 04:01:00 UTC
[jira] [Resolved] (ARROW-11735) [R] Allow Parquet and Arrow Dataset
to be optional components
[ https://issues.apache.org/jira/browse/ARROW-11735?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Neal Richardson resolved ARROW-11735.
-------------------------------------
Resolution: Fixed
Issue resolved by pull request 9610
[https://github.com/apache/arrow/pull/9610]
> [R] Allow Parquet and Arrow Dataset to be optional components
> -------------------------------------------------------------
>
> Key: ARROW-11735
> URL: https://issues.apache.org/jira/browse/ARROW-11735
> Project: Apache Arrow
> Issue Type: Sub-task
> Components: R
> Reporter: Neal Richardson
> Assignee: Ian Cook
> Priority: Major
> Labels: pull-request-available
> Fix For: 4.0.0
>
> Time Spent: 16h 50m
> Remaining Estimate: 0h
>
> 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)