deshanxiao commented on PR #1565: URL: https://github.com/apache/orc/pull/1565#issuecomment-1664968517
> BTW, do you know why [HADOOP-13126](https://issues.apache.org/jira/browse/HADOOP-13126) is still open? > It seems that the reviewer think this modification is too large to cause the Hadoop PR freeze. See https://github.com/apache/hadoop/pull/2723. > And, Hadoop JIRA seems to say the following. Could you double-check if there is no license issue or not? > > > [Brotli is licensed with the MIT license](https://github.com/google/brotli/blob/master/LICENSE), and the [JNI library jbrotli is ALv2](https://github.com/MeteoGroup/jbrotli/blob/master/LICENSE). From the dependencies: https://github.com/rdblue/brotli-codec/blob/9bbc2bcec1e7307e84552f28e90f20cb3099f189/build.gradle#L58 We are only indirectly referencing `jbrotli`. So the license requirement you mentioned is right. BTW, `brotli-codec` is just an adapter to meet with hadoop codec framework. We can use jbrotli directly. What do you think? @dongjoon-hyun -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
