[ 
https://issues.apache.org/jira/browse/FLINK-31167?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17692253#comment-17692253
 ] 

Matthias Pohl edited comment on FLINK-31167 at 2/22/23 3:44 PM:
----------------------------------------------------------------

[~SleePy] thanks for your explanation and the link to FLINK-7552. That makes 
actually sense. I'm still trying to get my head around the API backwards 
compatibility. That's why I'm asking to be on the safe side. One more thing: 
wouldn't we be more stricter by throwing an {{UnsupportedOperationException}} 
as the default behavior of the deprecated method implementation? Or am I 
missing some points here?


was (Author: mapohl):
[~SleePy] thanks for your explanation and the link to FLINK-7552. That makes 
actually sense. I'm still trying to get my head around the API backwards 
compatibility. That's why I'm asking to be on the safe side. One more thing:
wouldn't we be more stricter by throwing an {{UnsupportedOperationException}} 
as the default behavior of the deprecated method implementation? Or am I 
missing some points here?

> Verify that no exclusions were erroneously added to the japicmp plugin
> ----------------------------------------------------------------------
>
>                 Key: FLINK-31167
>                 URL: https://issues.apache.org/jira/browse/FLINK-31167
>             Project: Flink
>          Issue Type: Sub-task
>            Reporter: Matthias Pohl
>            Assignee: Matthias Pohl
>            Priority: Major
>
> Verify that no exclusions were erroneously added to the japicmp plugin that 
> break compatibility guarantees. Check the exclusions for the 
> japicmp-maven-plugin in the root pom (see 
> [apache/flink:pom.xml:2175ff|https://github.com/apache/flink/blob/3856c49af77601cf7943a5072d8c932279ce46b4/pom.xml#L2175]
>  for exclusions that:
> * For minor releases: break source compatibility for {{@Public}} APIs
> * For patch releases: break source/binary compatibility for 
> {{@Public}}/{{@PublicEvolving}}  APIs
> Any such exclusion must be properly justified, in advance.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to