-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GH-5: [Docs] Add LICENSE and NOTICE files #39
Conversation
I've trimmed down and removed all dependencies that seem to refer to C++ / Python / and other implementations and left the ones that are related to Java (or seem to be related to Java). |
LICENSE.txt
Outdated
-------------------------------------------------------------------------------- | ||
|
||
This product includes software from Dremio, which is made available under the | ||
Apache License 2.0. | ||
* Copyright (C) 2017-2018 Dremio Corporation | ||
* https://github.com/dremio/dremio-oss |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It seems that we can remove this because we removed "Copyright (C) 2017-2018 Dremio Corporation" by apache/arrow#5602 .
Anyway, I should have asked why we can remove it when I merge...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kou ! I've removed it
…resent on this repo
Co-authored-by: Sutou Kouhei <kou@cozmixng.org>
I've rebased main but it seems pre-commit is failing there. I've opened: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Closes #5