Github user srowen commented on a diff in the pull request:

    https://github.com/apache/spark/pull/1813#discussion_r16034200
  
    --- Diff: core/src/main/java/com/google/common/base/Optional.java ---
    @@ -0,0 +1,243 @@
    +/*
    + * Copyright (C) 2011 The Guava Authors
    --- End diff --
    
    Not according to my reading of
    http://www.apache.org/dev/licensing-howto.html , in the part covering
    bundled AL2 dependencies, but this stuff is always less than intuitive to
    me.
    
    I suppose the reasoning is that Spark's AL2 license already exactly
    describes the terms of the licensing for that file. That's not quite true
    for BSD or MIT licenses.
    
    The copyright owner doesn't matter; there are hundreds of them. It matters
    just how whoever they are license their IP to be used.
    On Aug 11, 2014 12:22 AM, "Matei Zaharia" <[email protected]> wrote:
    
    > In core/src/main/java/com/google/common/base/Optional.java:
    >
    > > @@ -0,0 +1,243 @@
    > > +/*
    > > + * Copyright (C) 2011 The Guava Authors
    >
    > Shouldn't we still list it in our LICENSE? We've listed all source files
    > that are not copyright Apache, which would include this one.
    >
    > —
    > Reply to this email directly or view it on GitHub
    > <https://github.com/apache/spark/pull/1813/files#r16034129>.
    >


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to