Skip to content

Conversation

@vanzin
Copy link
Contributor

@vanzin vanzin commented Aug 10, 2015

If the correct parameter is not provided, Hive will run into an error
because it calls methods that are specific to the local filesystem to
copy the data.

If the correct parameter is not provided, Hive will run into an error
because it calls methods that are specific to the local filesystem to
copy the data.
@vanzin
Copy link
Contributor Author

vanzin commented Aug 10, 2015

I tested this on an actual cluster; not sure whether we can write a unit test (since we don't run an HDFS mini server in our tests).

@SparkQA
Copy link

SparkQA commented Aug 11, 2015

Test build #40349 has finished for PR 8086 at commit 5acad2b.

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

@vanzin
Copy link
Contributor Author

vanzin commented Aug 11, 2015

/cc @marmbrus

@vanzin
Copy link
Contributor Author

vanzin commented Aug 11, 2015

/cc @yhuai

@marmbrus
Copy link
Contributor

Thanks, merging to master and 1.5

asfgit pushed a commit that referenced this pull request Aug 12, 2015
If the correct parameter is not provided, Hive will run into an error
because it calls methods that are specific to the local filesystem to
copy the data.

Author: Marcelo Vanzin <vanzin@cloudera.com>

Closes #8086 from vanzin/SPARK-9804.

(cherry picked from commit 57ec27d)
Signed-off-by: Michael Armbrust <michael@databricks.com>
@asfgit asfgit closed this in 57ec27d Aug 12, 2015
CodingCat pushed a commit to CodingCat/spark that referenced this pull request Aug 17, 2015
If the correct parameter is not provided, Hive will run into an error
because it calls methods that are specific to the local filesystem to
copy the data.

Author: Marcelo Vanzin <vanzin@cloudera.com>

Closes apache#8086 from vanzin/SPARK-9804.
@vanzin vanzin deleted the SPARK-9804 branch August 17, 2015 22:23
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.

3 participants