eric-maynard commented on code in PR #1925:
URL: https://github.com/apache/polaris/pull/1925#discussion_r2164564217


##########
spec/polaris-management-service.yml:
##########
@@ -911,6 +911,7 @@ components:
             - OAUTH
             - BEARER
             - SIGV4
+            - NO_AUTH

Review Comment:
   Sorry, what do you mean by the other way around? Does this relate to the doc 
linked above?
   
   edit: Oh, are you referring to what happens if a client tries to provide 
this new enum value to an older version of the service? How is that different 
from a client calling an API that doesn't exist on the service? I don't think 
that typically qualifies as a breaking change under semver
   
   Regardless of whether or not it is a breaking change however, we do need to 
discuss the spec change on the ML just like the linked PR



-- 
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]

Reply via email to