jerrypeng commented on pull request #6983:
URL: https://github.com/apache/pulsar/pull/6983#issuecomment-631748364
@srkukarni thinking more about it, I am wondering for sources and sinks
whether the following interface is appropriate now that we formalized the
concept of a connector config.
```
void open(final Map<String, Object> config, SourceContext sourceContext)
throws Exception;
```
Since a connector will have a config class attached to it should we still
have "Map<String, Object> config" as an argument or should we somehow just pass
in the config POJO?
----------------------------------------------------------------
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]
