You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/06/01 15:32:41 UTC

[GitHub] [pulsar] sijie commented on pull request #10697: [fix #9640] remove `pulsar-client-admin-api` dependency : `pulsar-common`

sijie commented on pull request #10697:
URL: https://github.com/apache/pulsar/pull/10697#issuecomment-852221913


   My take here: 
   
   We are in a situation that we can’t roll back any of this change *easily. This change involves many PRs from many modules including client, client-admin, and functions. The earliest PR related to this change is probably a couple of months ago. So we need this change to land for 2.8.0
   
   It is already a breaking change that we need to accept for pulsar-client-admin. It is hard to get to a very clean and ideal state in one release. I see what the PR proposed is the *best middle-ground situation that we can get between the 2.8.0 release and subsequent releases.
   
   To unblock the 2.8.0 release, I think we can add @Evolving annotation to the pulsar client admin API to indicate there will be subsequent changes.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org