flyrain commented on code in PR #3759: URL: https://github.com/apache/polaris/pull/3759#discussion_r2819963048
########## CHANGELOG.md: ########## @@ -32,6 +32,9 @@ request adding CHANGELOG notes for breaking (!) changes and possibly other secti ### Upgrade notes - The custom token-bucket based rate limiter has been replaced with Guava's rate limiter implementation. +- The Helm chart now includes a JSON schema file for easy validation of values files. Because types + are now validated, existing values files may need to be updated to match the new schema. Review Comment: Should we consider it as a breaking change? ########## CHANGELOG.md: ########## @@ -32,6 +32,9 @@ request adding CHANGELOG notes for breaking (!) changes and possibly other secti ### Upgrade notes - The custom token-bucket based rate limiter has been replaced with Guava's rate limiter implementation. +- The Helm chart now includes a JSON schema file for easy validation of values files. Because types + are now validated, existing values files may need to be updated to match the new schema. + Review Comment: Nit: extra empty line -- 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]
