Skip to content

Conversation

@liancheng
Copy link
Contributor

Path.toUri.getPath strips scheme part of output path (from file:///foo to /foo), which causes ORC data source only writes to the file system configured in Hadoop configuration. Should use Path.toString instead.

@liancheng
Copy link
Contributor Author

cc @zhzhan

@SparkQA
Copy link

SparkQA commented Jun 19, 2015

Test build #35200 has finished for PR 6892 at commit 87f8199.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@liancheng
Copy link
Contributor Author

Merging to master and branch-1.4.

asfgit pushed a commit that referenced this pull request Jun 19, 2015
…g ORC files

`Path.toUri.getPath` strips scheme part of output path (from `file:///foo` to `/foo`), which causes ORC data source only writes to the file system configured in Hadoop configuration. Should use `Path.toString` instead.

Author: Cheng Lian <[email protected]>

Closes #6892 from liancheng/spark-8458 and squashes the following commits:

87f8199 [Cheng Lian] Don't strip scheme of output path when writing ORC files

(cherry picked from commit a71cbbd)
Signed-off-by: Cheng Lian <[email protected]>
@asfgit asfgit closed this in a71cbbd Jun 19, 2015
@liancheng liancheng deleted the spark-8458 branch June 19, 2015 05:24
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
…g ORC files

`Path.toUri.getPath` strips scheme part of output path (from `file:///foo` to `/foo`), which causes ORC data source only writes to the file system configured in Hadoop configuration. Should use `Path.toString` instead.

Author: Cheng Lian <[email protected]>

Closes apache#6892 from liancheng/spark-8458 and squashes the following commits:

87f8199 [Cheng Lian] Don't strip scheme of output path when writing ORC files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants