pan3793 commented on PR #5686: URL: https://github.com/apache/kyuubi/pull/5686#issuecomment-2026876821
@bkyryliuk would you mind making the following changes? Then we can merge this patch. 1. rename the top-level dir `pyhive` to `python` 2. add `python/**` into `.rat-exlucdes` to temporarily skip the license header check, we can fix it later. 3. as we discussed in the mailing list before, you may want to update `NOTICE` to add such info ```patch Apache Kyuubi Copyright 2021-2024 The Apache Software Foundation. This product includes software developed at The Apache Software Foundation (https://www.apache.org/). + (This is an example, please tune the words by yourself) + Portions of this software were developed at Dropbox, Inc. ... ``` -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
