I just realized that the current approach in the PR has a troublesome flaw: the default API is actually mixed iff a newer version defines a new url. (if we add `/totally/new/url` in `v2` then the default for other urls would be `v1`, but for the new one it would be `v2`). As you suggested we should restrict the default to a specific version, which would also solve the issue of multiple registrations.
What we could do is add a `isDefault` flag to the `RestAPIVersion` enum, and only register unversioned urls for that version. We could enforce with a test that exactly one version is marked as the default. WDYT? [ Full content available at: https://github.com/apache/flink/pull/6602 ] This message was relayed via gitbox.apache.org for [email protected]
