nicusX commented on code in PR #1:
URL: 
https://github.com/apache/flink-connector-prometheus/pull/1#discussion_r1633781977


##########
README.md:
##########
@@ -8,6 +8,14 @@ Apache Flink is an open source stream processing framework 
with powerful stream-
 
 Learn more about Flink at 
[https://flink.apache.org/](https://flink.apache.org/)
 
+## Modules
+
+This repository contains the following modules
+
+* [Prometheus Connector](./prometheus-connector): Flink Prometheus Connector 
implementation; supports optional request signer
+* [Sample application](./example-datastream-job): Sample application showing 
the usage of the connector with DataStream API. It also demonstrates how to 
configure the request signer. 

Review Comment:
   I moved the example application under tests, as suggested, merging 
everything in a single class.
   The usage of the AMP connector is commented out, being a separate 
dependency, but still present to show the usage, in case someone wants to use 
it.
   
   
https://github.com/apache/flink-connector-prometheus/blob/5afb890aa8fe860ce25bcf89f7cdee93a6d2c4be/flink-connector-prometheus/src/test/java/org/apache/flink/connector/prometheus/sink/examples/DataStreamExample.java
 



-- 
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]

Reply via email to