flyrain commented on PR #1197:
URL: https://github.com/apache/polaris/pull/1197#issuecomment-2735022910
> LGTM ! Sidenote, they are public classes, do we have a deprecation model ?
We discussed a bit before, but I don't think we have any formal conclusion
yet. I'd suggest to follow similar approaches as other projects(e.g., Iceberg)
by leveraging semantic versioning to deprecate public interface. For example,
we can specify the version when we remove a interface like following. It seems
a big topic though. Feel free to bring it up in the dev mail list for
discussion.
```
/**
* @deprecated will be removed in 2.0.0
*/
@Deprecated
public void foo();
```
--
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]
