[ 
https://issues.apache.org/jira/browse/FLINK-17036?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17078077#comment-17078077
 ] 

Chesnay Schepler commented on FLINK-17036:
------------------------------------------

OK, then I'll take it from here. The PR went in the right direction (which is 
rather impressive (y)), but there were naturally a few things to clean up.

I will try it out later, and if it works merge it to be included in 1.11.0.

> Datadog forwarder does not allow for EU endpoint
> ------------------------------------------------
>
>                 Key: FLINK-17036
>                 URL: https://issues.apache.org/jira/browse/FLINK-17036
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / Metrics
>            Reporter: Raymond Farrelly
>            Assignee: Raymond Farrelly
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 1.11.0
>
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> Please note I am a current Datadog employee. 
> [https://github.com/apache/flink/pull/11661]
> Essentially the FLINK integration does not work in the EU instance due to:
> [https://github.com/apache/flink/blob/master/flink-metrics/flink-metrics-datadog/src/main/java/org/apache/flink/metrics/datadog/DatadogHttpClient.java#L45]
> {{`.com` is hardcoded. Datadog has two endpoints `.com` and `.eu` both need 
> to be accounted for or EU customers cannot use the integration. 
> [https://docs.datadoghq.com/agent/basic_agent_usage/?tab=agentv6v7#datadog-site]}}
> As a quick suggestion I put together a PR that would account for this. Please 
> note I am not a software Engineer so this is very much a starting point.
> I followed the below to ensure it builds:
> {{```}}
>  {{ git clone [https://github.com/apache/flink.git]}}
>  \{{ cd flink}}
>  \{{ mvn clean package -DskipTests # this will take up to 10 minutes}}
>  \{{ ```}}
> {{Made some changes and did a rough build on this which was successful:}}
> {\{ ```}}
>  \{{ raymond.farrelly@RFM flink % mvn clean -pl :flink-metrics-datadog}}
>  \{{ ...}}
>  {{ [INFO] }}
>  {{ [INFO] ---------------< org.apache.flink:flink-metrics-datadog 
> >---------------}}
>  {{ [INFO] Building flink-metrics-datadog 1.11-SNAPSHOT}}
>  {{ [INFO] --------------------------------[ jar 
> ]---------------------------------}}
>  {{ [INFO] }}
>  {{ [INFO] — maven-clean-plugin:3.1.0:clean (default-clean) @ 
> flink-metrics-datadog —}}
>  {{ [INFO] Deleting 
> /Users/raymond.farrelly/Documents/flink/flink-metrics/flink-metrics-datadog/target}}
>  {{ [INFO] 
> ------------------------------------------------------------------------}}
>  {{ [INFO] BUILD SUCCESS}}
>  {{ [INFO] 
> ------------------------------------------------------------------------}}
>  {{ [INFO] Total time: 1.292 s}}
>  {{ [INFO] Finished at: 2020-04-02T18:06:18+01:00}}
>  {{ [INFO] 
> ------------------------------------------------------------------------}}
>  \{{ raymond.farrelly@RFM flink % }}
>  {
> { ```}
> }



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to