[ 
https://issues.apache.org/jira/browse/SPARK-1629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13980893#comment-13980893
 ] 

Sean Owen commented on SPARK-1629:
----------------------------------

Oh I see, that was added yesterday and I hadn't updated my fork. Yes that's 
right. It only works since commons-lang happens to be imported by another 
dependency. I don't think it's worth bringing it all in just for one method.

How about we just 'inline' this with a utility method like:

{code:scala}
def isWindows(): Boolean = {
  try {
    val osName = System.getProperty("os.name")
    osName != null && osName.startsWith("Windows")
  } catch {
    case e: SecurityException => (log a warning and return false)
  }
}
{code}


>  Spark Core missing commons-lang dependence
> -------------------------------------------
>
>                 Key: SPARK-1629
>                 URL: https://issues.apache.org/jira/browse/SPARK-1629
>             Project: Spark
>          Issue Type: Bug
>            Reporter: witgo
>




--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to