[
https://issues.apache.org/jira/browse/MESOS-3023?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14631537#comment-14631537
]
Marco Massenzio commented on MESOS-3023:
----------------------------------------
Thanks for doing this!
I haven't looked at the review, but I'm aware of some conversations about
creating a {{URI}} class in {{stout}}.
That one could have a static factory method.
If this is exactly what you have done (sorry, on a train, not easy to browse
around) I'm sure you'll find it easy to get either of [~vinodkone] or
@benmahler to shepherd this :)
> Factoring out the pattern for URL generation
> ---------------------------------------------
>
> Key: MESOS-3023
> URL: https://issues.apache.org/jira/browse/MESOS-3023
> Project: Mesos
> Issue Type: Task
> Reporter: Artem Harutyunyan
> Assignee: Klaus Ma
> Priority: Minor
> Labels: beginner, mesosphere, newbie
>
> fetcher_test.cpp uses the following code for generating URLs:
> string url = "http://"; + net::getHostname(process.self().address.ip).get() +
> ":" + stringify(process.self().address.port) + "/" + process.self().id
> it would be good to isolate that code in a function, and replace the code
> above with something like:
> string url = "http://"; + endpoint_url(process, "uri_test");
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
