[GitHub] snleee commented on issue #3722: Update LICENSE and NOTICE files

2019-01-28 Thread GitBox
snleee commented on issue #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-458420180 @felixcheung The current pr includes the license with MIT and BSD under `/licenses`. If you don't have any other concerns, would you give me a shi

[GitHub] snleee commented on issue #3722: Update LICENSE and NOTICE files

2019-01-22 Thread GitBox
snleee commented on issue #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-456516171 > My take is if it's MIT license it needs not be included: > http://www.apache.org/dev/licensing-howto.html > http://www.apache.org/dev/licen

[GitHub] snleee commented on issue #3722: Update LICENSE and NOTICE files

2019-01-21 Thread GitBox
snleee commented on issue #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-456220187 @kishoreg Another approach that we can take is to copy licenses into `LICENSE` file. http://www.apache.org/dev/licensing-howto.html#permissive-dep

[GitHub] snleee commented on issue #3722: Update LICENSE and NOTICE files

2019-01-18 Thread GitBox
snleee commented on issue #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-455721410 @felixcheung Can you check this when you find the time? This is an automated mess

[GitHub] snleee commented on issue #3722: Update LICENSE and NOTICE files

2019-01-18 Thread GitBox
snleee commented on issue #3722: Update LICENSE and NOTICE files URL: https://github.com/apache/incubator-pinot/pull/3722#issuecomment-455720913 After removing category X dependency and removing "pinot-perf" from distribution package, we are bundling `109` external jar files and `9` js/css