Dominic Hamon created MESOS-2314:
------------------------------------
Summary: remove unnecessary constants
Key: MESOS-2314
URL: https://issues.apache.org/jira/browse/MESOS-2314
Project: Mesos
Issue Type: Improvement
Components: slave, technical debt
Reporter: Dominic Hamon
Priority: Minor
In {{src/slave/paths.cpp}} a number of string constants are defined to describe
the formats of various paths. However, given there is a 1:1 mapping between the
string constant and the functions that build the paths, the code would be more
readable if the format strings were inline in the functions.
In the cases where one constant depends on another (see the
{{EXECUTOR_INFO_PATH, EXECUTOR_PATH, FRAMEWORK_PATH, SLAVE_PATH, ROOT_PATH}}
chain, for example) the function calls can just be chained together.
This will have the added benefit of removing some statically constructed string
constants, which are dangerous.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
