[
https://issues.apache.org/jira/browse/MESOS-116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13186105#comment-13186105
]
[email protected] commented on MESOS-116:
-----------------------------------------------------
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/3498/
-----------------------------------------------------------
(Updated 2012-01-14 05:42:06.560262)
Review request for mesos, Benjamin Hindman and Andy Konwinski.
Summary (updated)
-------
Moved configuration setup from the main.cpp files in the slave and master
directories to the registerOption functions in slave.cpp and master.cpp. Made
the default listening port and WebUI ports constants, and made the constants
marked with TODOs in master/constants.hpp configurable. I made a git patch
because I wasn't sure if the review board accepts svn patches (the one I
uploaded to the JIRA was svn).
This addresses bug MESOS-116.
https://issues.apache.org/jira/browse/MESOS-116
Diffs
-----
src/master/constants.hpp 8248475
src/master/main.cpp e427ff7
src/master/master.cpp b56c732
src/slave/constants.hpp f0c8679
src/slave/main.cpp 9519ed2
src/slave/slave.cpp 62fc277
Diff: https://reviews.apache.org/r/3498/diff
Testing
-------
Thanks,
Sam
> Make default configuration values constants and make certain constants
> configurable
> -----------------------------------------------------------------------------------
>
> Key: MESOS-116
> URL: https://issues.apache.org/jira/browse/MESOS-116
> Project: Mesos
> Issue Type: Improvement
> Reporter: Sam Pullman
> Priority: Minor
> Labels: configuration
> Attachments: MESOS-116.patch
>
> Original Estimate: 24h
> Remaining Estimate: 24h
>
> Change hard-coded default configuration values in master and slave setup to
> be constants. Update configuration options to include constants in master
> marked with TODOs.
--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators:
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira
