featzhang commented on PR #30: URL: https://github.com/apache/flink-connector-http/pull/30#issuecomment-4257487167
Hi @davidradl and @mariyageorge01, You have both previously contributed to the HTTP connector documentation ([`docs/content/docs/connectors/table/http.md`](https://github.com/apache/flink-connector-http/blob/main/docs/content/docs/connectors/table/http.md)). This PR adds a **Quick Start** section to help new users get started quickly with SQL and DataStream API examples. I have addressed all previous review comments, and the latest commits (pushed on 2026-03-31) have: - Reorganized content to follow the style used by other Flink connectors (similar to the Kafka connector docs) - Fixed all connector identifiers (`http-async-sink` → `http-sink` etc.) - Added SQL examples for HTTP Sink and Lookup Source - Added DataStream API (Java) example for `HttpSink` - Added a configuration options reference table CI is currently **passing** ✅ on both Java 11 and Java 17. Could you please take a look and share your feedback? PTAL, thanks! 🙏 -- 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]
