Neil Conway created MESOS-3772:
----------------------------------

             Summary: Consistency of quoted strings in error messages
                 Key: MESOS-3772
                 URL: https://issues.apache.org/jira/browse/MESOS-3772
             Project: Mesos
          Issue Type: Bug
            Reporter: Neil Conway


Example log output:

{quote}
I1020 18:56:02.933956  1790 slave.cpp:1270] Got assigned task 13 for framework 
496620b9-4368-4a71-b741-68216f3d909f-0000
I1020 18:56:02.934185  1790 slave.cpp:1386] Launching task 13 for framework 
496620b9-4368-4a71-b741-68216f3d909f-0000
I1020 18:56:02.934408  1790 slave.cpp:1618] Queuing task '13' for executor 
default of framework '496620b9-4368-4a71-b741-68216f3d909f-0000
I1020 18:56:02.935417  1790 slave.cpp:1760] Sending queued task '13' to 
executor 'default' of framework 496620b9-4368-4a71-b741-68216f3d909f-0000
{quote}

Aside from the typo (unmatched quote) in the third line, these log messages 
using quoting inconsistently: sometimes task, executor, and framework IDs are 
quoted, other times they are not.

We should probably adopt a general rule, a la 
http://www.postgresql.org/docs/9.4/static/error-style-guide.html . My proposal: 
when interpolating a variable, only use quotes if it is possible that the value 
might contain whitespace or punctuation (in the latter case, the punctuation 
should probably be escaped).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to