darenwkt commented on PR #22: URL: https://github.com/apache/flink-connector-prometheus/pull/22#issuecomment-2848616292
> Thanks for the PR I am not sure I fully understand the Table API interface and whether it matches the use case this connector is supposed to be used for. I left some specific comments about the API. Would you please clarify? > > Also, I am concerned of breaking the public interface of DataStream API. Is that required? Thanks Lorenzo for the review, I have addressed comments on the API use case and example and I hope that it clarifies it better now. I have also found a way to support TableAPI without breaking changes, appreciate if you can take another look at the updated PR, thank you -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
