You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@mesos.apache.org by "Alexander Rukletsov (JIRA)" <ji...@apache.org> on 2018/08/08 14:04:00 UTC

[jira] [Assigned] (MESOS-3772) Consistency of quoted strings in error messages

     [ https://issues.apache.org/jira/browse/MESOS-3772?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Alexander Rukletsov reassigned MESOS-3772:
------------------------------------------

    Assignee:     (was: Isabel Jimenez)

> 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
>            Priority: Major
>              Labels: mesosphere, newbie, tech-debt
>
> 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
(v7.6.3#76005)