mesos-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Marco Massenzio (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MESOS-3023) Factoring out the pattern for URL generation
Date Fri, 17 Jul 2015 16:28:05 GMT

    [ 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)

Mime
View raw message