[
https://issues.apache.org/jira/browse/SPARK-18093?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Mark Grover updated SPARK-18093:
--------------------------------
Description:
At least on my mac (with JDK 1.7.0_67), {{default value of WAREHOUSE_PATH}} in
SQLConfSuite fails because left side of the assert doesn't have a trailing
slash while the right does.
As [~srowen] mentions
[here|https://github.com/apache/spark/pull/15382#discussion_r84240197], the JVM
adds a trailing slash if the directory exists and doesn't if it doesn't. I
think it'd be good for the test to work regardless of the directory's existence.
was:
At least on my mac (with JDK 1.7.0_67), {{default value of WAREHOUSE_PATH}}
fails because left side of the assert doesn't have a trailing slash while the
right does.
As [~srowen] mentions
[here|https://github.com/apache/spark/pull/15382#discussion_r84240197], the JVM
adds a trailing slash if the directory exists and doesn't if it doesn't. I
think it'd be good for the test to work regardless of the directory's existence.
> Fix default value test in SQLConfSuite to work regardless of warehouse dir's
> existence
> --------------------------------------------------------------------------------------
>
> Key: SPARK-18093
> URL: https://issues.apache.org/jira/browse/SPARK-18093
> Project: Spark
> Issue Type: Bug
> Components: SQL
> Affects Versions: 2.0.2, 2.1.0
> Reporter: Mark Grover
>
> At least on my mac (with JDK 1.7.0_67), {{default value of WAREHOUSE_PATH}}
> in SQLConfSuite fails because left side of the assert doesn't have a trailing
> slash while the right does.
> As [~srowen] mentions
> [here|https://github.com/apache/spark/pull/15382#discussion_r84240197], the
> JVM adds a trailing slash if the directory exists and doesn't if it doesn't.
> I think it'd be good for the test to work regardless of the directory's
> existence.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
