adutra commented on PR #2877: URL: https://github.com/apache/polaris/pull/2877#issuecomment-3448513576
> we are just saying hey this is something we have, try it out if you want to, if we put out a breaking change we should call it out in our release docs, reconsider that there is users using it and make change accordingly and this is true for any features we ship. The above is not dishonest per se, but it's omitting two important messages: 1. The API is still evolving, including the SQL schema. 2. The implementation is currently incomplete. > I propose i remove the CloudWatch sink from the release notes entirely [...] if we want we can mark CloudWatch sink or the whole EventListner in general as beta in the follow-up. AFAIK, _both_ sinks (CloudWatch and SQL) are being introduced in 1.2. Regardless of any mention in the changelog, this means that if we don't mark the API experimental now, they will be implicitly released as production-ready. It would be strange to downgrade this API from production-ready to beta in a future release imho. -- 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]
