Uploaded image for project: 'WiredTiger'
  1. WiredTiger
  2. WT-753

more info in messages

    XMLWordPrintable

    Details

    • Type: Task
    • Status: Closed
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: WT1.6.6
    • Component/s: None
    • Labels:

      Description

      I find myself often wishing for or manually adding both a timestamp and pid:tid prefix for verbose messages. I think this is a generally useful thing and would like to add it. Any objections?

      I have several questions relating to this:

      • Should it just be for 'msg_event' messages (i.e. verbose) or all message types? Just verbose works well enough for debugging convenience. Having all types have the same prefixes seems desirable too.
      • Other than a comment in __eventv, the structure of a message seems like it is not documented and this wouldn't break things. Did I miss it?
      • The comment in __eventv separates things with a comma. Do we expect people are parsing these as comma-separated values? This should probably follow suit.
      • What should it look like? Any preferences? Two obvious candidates would be:
        sec:usec, pid:tid, session->error_prefix, ...
        [sec:usec], [pid:tid], session->error_prefix, ...

        Attachments

          Activity

            People

            • Assignee:
              sue.loverso Susan LoVerso
              Reporter:
              sue.loverso Susan LoVerso
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: