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

ASF GitHub Bot commented on PARQUET-2371:
-----------------------------------------

gszadovszky commented on PR #1181:
URL: https://github.com/apache/parquet-mr/pull/1181#issuecomment-1802279769

   Sorry @wgtmac, @ConeyLiu, for the late response.
   I don't think it is a bug in japicmp to fail with a new method in an 
existing interface. It would fail a 3rd party if it implements the interface. 
To avoid keep excluding files we might add a default implementation for the new 
methods instead, if possible.




> Resolve japicmp failure for CI
> ------------------------------
>
>                 Key: PARQUET-2371
>                 URL: https://issues.apache.org/jira/browse/PARQUET-2371
>             Project: Parquet
>          Issue Type: Improvement
>          Components: parquet-mr
>            Reporter: Atour Mousavi Gourabi
>            Assignee: Atour Mousavi Gourabi
>            Priority: Major
>             Fix For: 1.14.0
>
>
> Japicmp triggers on newly added methods, which have not been changed. This is 
> likely a japicmp bug. Excludes should be added for the affected classes, as 
> otherwise the CI job will not run anything after the japicmp goal. See the 
> discussion at https://github.com/apache/parquet-mr/pull/1141.



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

Reply via email to