merlimat commented on pull request #10878: URL: https://github.com/apache/pulsar/pull/10878#issuecomment-858357896
>@merlimat the KeyValueSchema class is an important case. if we go this way I would like to add a new public interface that represents KeyValueSchema and is it usable by the Sink. That's fine (although I don't understand exactly why `Schema<KeyValue>` doesn't work there). Having said that, I think it's a completely different discussion from this PR. The expectation for functions and connectors is to have access to `pulsar-client-api` and not (directly) to `pulsar-client`. That has always been the case, even if through different implementation mechanism (first shading and now class loaders). -- 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]
