Skip to content

Unified Log Format RFC#18

Merged
huachaohuang merged 10 commits intomasterfrom
log-format-standard
Jan 7, 2019
Merged

Unified Log Format RFC#18
huachaohuang merged 10 commits intomasterfrom
log-format-standard

Conversation

@breezewish
Copy link
Copy Markdown
Member

No description provided.

Signed-off-by: Breezewish <breezewish@pingcap.com>
@breezewish

This comment has been minimized.

Comment thread text/2018-12-19-unified-log-format.md
Comment thread text/2018-12-19-unified-log-format.md
Comment thread text/2018-12-19-unified-log-format.md Outdated
Signed-off-by: Breezewish <breezewish@pingcap.com>
@Hoverbear Hoverbear added the Initial Comment Period This RFC is in the initial comment period, and has quite some time to give input on. label Dec 20, 2018
@breezewish
Copy link
Copy Markdown
Member Author

Comments addressed, mainly in the following aspects:

  • How to deal with non-UTF8 characters.
  • Make logging message as a field (and print it first).
  • How to deal with long fields.

@overvenus @BusyJay @lysu @siddontang @shenli PTAL, thanks!

Comment thread text/2018-12-19-unified-log-format.md Outdated
Signed-off-by: Breezewish <breezewish@pingcap.com>
@breezewish breezewish force-pushed the log-format-standard branch from f4fc44e to 47081f6 Compare January 7, 2019 04:28
Signed-off-by: Breezewish <breezewish@pingcap.com>
Signed-off-by: Breezewish <breezewish@pingcap.com>
@overvenus overvenus added Final Comment Period This RFC is in the final comment period, and has a limited amount of time to give input on. and removed Initial Comment Period This RFC is in the initial comment period, and has quite some time to give input on. labels Jan 7, 2019

Sample: `2018/12/15 14:20:11.015 +08:00`

- `LEVEL`: The log level in upper case. Available levels are `FATAL`, `ERROR`,
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need to add the definition / meaning of each level here?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nop, this RFC is only for formats. We have another internal doc stipulates what kind of logs are in what level.

Comment thread text/2018-12-19-unified-log-format.md Outdated
Comment thread text/2018-12-19-unified-log-format.md Outdated
Copy link
Copy Markdown

@zimulala zimulala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Signed-off-by: Breezewish <breezewish@pingcap.com>
Copy link
Copy Markdown

@AstroProfundis AstroProfundis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown
Contributor

@huachaohuang huachaohuang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@huachaohuang huachaohuang merged commit c6e4a22 into master Jan 7, 2019
@huachaohuang huachaohuang deleted the log-format-standard branch January 7, 2019 09:57
Hoverbear pushed a commit to sunxiaoguang/rfcs that referenced this pull request Feb 21, 2019
* unified log format

Signed-off-by: Breezewish <breezewish@pingcap.com>
Signed-off-by: Ana Hobden <operator@hoverbear.org>
Hoverbear pushed a commit to sunxiaoguang/rfcs that referenced this pull request Feb 21, 2019
* unified log format

Signed-off-by: Wenxuan Shi <breezewish@pingcap.com>
Signed-off-by: Ana Hobden <operator@hoverbear.org>
Hoverbear pushed a commit to sunxiaoguang/rfcs that referenced this pull request Feb 21, 2019
* unified log format

Signed-off-by: Wenxuan Shi <hi@breeswish.org>
Signed-off-by: Ana Hobden <operator@hoverbear.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Final Comment Period This RFC is in the final comment period, and has a limited amount of time to give input on.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants