You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2020/09/10 11:20:16 UTC

[GitHub] [zeppelin] kortemik opened a new pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

kortemik opened a new pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901


   …sion:hadoop-lzo:jar:0.4.19, see https://www.gnu.org/licenses/quick-guide-gplv3.html
   
   ### What is this PR for?
   Change license to be compatible with license of com.hadoop.gplcompression:hadoop-lzo:jar:0.4.19
   https://github.com/twitter/hadoop-lzo/blob/master/src/main/java/com/hadoop/compression/lzo/CChecksum.java#L4-L7
   
   see https://www.gnu.org/licenses/quick-guide-gplv3.html
   
   ### What type of PR is it?
   [Bug Fix | Improvement | Feature | Documentation | Hot Fix | Refactoring]
   Hot Fix
   
   ### Todos
   * [ ] - Task
   
   ### What is the Jira issue?
   https://issues.apache.org/jira/browse/ZEPPELIN-5033
   
   ### How should this be tested?
   Purpose of this pull request is to deepen the discussion needed regarding this issue.
   
   ### Screenshots (if appropriate)
   
   ### Questions:
   * Does the licenses files need update?
   ```Yes```
   * Is there breaking changes for older versions?
   * Does this needs documentation?
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] kortemik commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
kortemik commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690992968






----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] kortemik commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
kortemik commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690992968


   There we have it https://github.com/twitter/scalding/pull/1932 (:


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] Reamer commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
Reamer commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690297171


   You are right.
   By the way, Scalding itself is licensed under the Apache License 2.0. https://github.com/twitter/scalding/blob/develop/LICENSE
   The library was added with the commit in scaling. https://github.com/twitter/scalding/commit/92a633db10eeac02a0faa6413a5969e85452381e
   
   I am far from being a licensing expert, but maybe you can address a similar problem in the Scaling project.
   
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] kortemik commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
kortemik commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690992968


   There we have it https://github.com/twitter/scalding/pull/1932 (:


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] kortemik commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
kortemik commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690992968






----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] Reamer commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
Reamer commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690220175


   Hi @kortemik,
   Can you give us a hint where the Zeppelin integrates Hadoop-lzo?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [zeppelin] kortemik commented on pull request #3901: Change license to be compatible with license of com.hadoop.gplcompres…

Posted by GitBox <gi...@apache.org>.
kortemik commented on pull request #3901:
URL: https://github.com/apache/zeppelin/pull/3901#issuecomment-690222802


   Hi,
   
   I think Scalding interpreter pulls it as a dependency:
   [INFO] Including com.hadoop.gplcompression:hadoop-lzo:jar:0.4.19 in the shaded jar.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org