Github user bzz commented on the pull request:
https://github.com/apache/incubator-zeppelin/pull/869#issuecomment-222390840
@Leemoonsoo good point, and Py4j is already a part of out binary release fo
some time, so its been alredy listed in
'zeppelin-distribution/src/bin_license/LICENSE', only license text was missing.
@hriviere good catch about a license text, thank you for taking care.
Sorry for a confusion, but indeed as soon as this PR do not include any
source code copied form py4j itself - there is no need to add it to LICENSE
file, as it tcove only source code releases.
Could you update it again please?
Otherwise it looks great to me - we should be good to merge right after
final LICENSE update and moving license text to
'zeppelin-distributio/src/bin_license/licenses'
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
