You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pinot.apache.org by GitBox <gi...@apache.org> on 2019/01/21 23:04:54 UTC

[GitHub] snleee commented on a change in pull request #3722: Update LICENSE and NOTICE files

snleee commented on a change in pull request #3722: Update LICENSE and NOTICE files
URL: https://github.com/apache/incubator-pinot/pull/3722#discussion_r249599293
 
 

 ##########
 File path: LICENSE
 ##########
 @@ -199,4 +198,170 @@
    distributed under the License is distributed on an "AS IS" BASIS,
    WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
    See the License for the specific language governing permissions and
-   limitations under the License.
\ No newline at end of file
+   limitations under the License.
+
+------------------------------------------------------------------------------------
+This project bundles some components that are also licensed under the Apache
+License Version 2.0:
+
+com.101tec:zkclient
 
 Review comment:
   I just checked Spark's source code and binary distribution. For the source code, they include `LINCENSE, NOTICE` file for source code distribution while they include `LICENCE-binary, NOTICE-binary` for their binary distribution. I will keep 2 different license files one for source code and the other for binary.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pinot.apache.org
For additional commands, e-mail: dev-help@pinot.apache.org