nlu90 opened a new pull request #10631:
URL: https://github.com/apache/pulsar/pull/10631


   ### Motivation
   
   Currently the context relationship for function, source and sink is not well 
defined. This prevents some common features to be added once for all and 
creates some confusion, code duplication in the current repo. As demonstrated 
in the following graph, this PR changes the hierarchy from left to right. By 
introducing a common base context, it help solving some issues we are seeing. 
The base context provides common access to pulsar cluster, state, metrics, and 
meta-data to make sure all components can reuse it.
   
   ![context 
hierarchy](https://user-images.githubusercontent.com/16407807/118730483-8ebf5200-b7ec-11eb-9220-d41261f148bb.png)
   
   
   
   ### Modifications
   
   - Remove `ConnectorContext` interface.
   - Introduce a `BaseContext` interface. 
   - Update existing `Context`, `SourceContext`, `SinkContext` interface to 
extend the new common interface.
   
   ### Verifying this change
   
   - [ ] Make sure that the change passes the CI checks.
   
   
   This change is a trivial rework / code cleanup without any test coverage.
   
   ### Does this pull request potentially affect one of the following parts:
   
   *If `yes` was chosen, please highlight the changes*
   
     - Dependencies (does it add or upgrade a dependency): (yes / no)
     - **The public API:**
   


-- 
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:
[email protected]


Reply via email to