[GitHub] [pulsar] eolivelli commented on pull request #11056: [pulsar-io] pass pulsar client via context to connector

[GitBox <gi...@apache.org>]

eolivelli commented on pull request #11056:
URL: https://github.com/apache/pulsar/pull/11056#issuecomment-869390714


   @jerrypeng thanks for your clarification. 
   
   @sijie 
   I am totally fine with adding this feature, and also I am very happy to see the Debezium problem fixed.
   
   But my point is that here we are adding again a new powerful API to the Pulsar Functions/IO system.
   **So this kind of changes in my humble option MUST be discussed properly on the dev@ mailing list.**
   This way the project/community will be aware of what's going on and have the chance to participate into the discussion
   for a new API.
   A PR is not good for such high level discussions, because not everyone is following the huge quantity of messages due to PR.
   
   We said many times during the past months (cc @merlimat) that we should use better the tool of the PIPs and we should add new APIs more carefully, being sure that the community is aware of what is happening. 
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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