julianhyde commented on pull request #2622: URL: https://github.com/apache/calcite/pull/2622#issuecomment-996981403
Regarding category X. I accept that we can use category X licensed components in testing. However, they are radioactive, and I think they should be labeled as category X in the source code. Without the 'Category X' comment, it would be easy for someone to change `testImplementation` to `implementation`. It's especially worth calling out because not everyone knows that Elasticsearch has changed license. Regarding repositories. I don't think it's worth the pain and fragility of adding a new repository, even for one release, and even with gradle guard rails that limit the impact to just this component. Anyone who is writing serious open source Java code needs to get it into Maven Central. A hundred IT people are going to have to add a new repository to their corporate Maven mirror, cursing Calcite for being high maintenance. How much do we need to back out or downgrade in order to do without the repository? -- 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]
