Uploaded image for project: 'Apache Arrow'
  1. Apache Arrow
  2. ARROW-7664

[C++] Extract localfs default from FileSystemFromUri

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 0.15.1
    • 0.17.0
    • C++

    Description

      https://github.com/apache/arrow/pull/6257#pullrequestreview-347506792

      The argument to FileSystemFromUri should always be rfc3986 formatted. The current fallback to localfs can be recovered by adding static string Uri::FromPath(string) which wraps uriWindowsFilenameToUriStringA and the corresponding unix path function.

      FileSystemFromUri(Uri::FromPath(R"(E:\dir\file.txt)"), &sanitized) 

      This is a little more boilerplate but I think it's worthwhile to be explicit here.

      Attachments

        Issue Links

          Activity

            People

              apitrou Antoine Pitrou
              bkietz Ben Kietzman
              Votes:
              0 Vote for this issue
              Watchers:
              3 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 - 2h 50m
                  2h 50m