Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-10734 [R] Build and test on Solaris
  3. ARROW-11735

[R] Allow Parquet and Arrow Dataset to be optional components

    XMLWordPrintableJSON

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 4.0.0
    • Component/s: R

      Description

      Parquet requires thrift and it seems that thrift (at least as of version 0.12) does not compile on Solaris:

      /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) {
      

      (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.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                icook Ian Cook
                Reporter:
                npr Neal Richardson
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 17h 10m
                  17h 10m