Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

*: log format for tools, client, pkg, table #1424

Merged
merged 3 commits into from
Feb 18, 2019

Conversation

rleungx
Copy link
Member

@rleungx rleungx commented Jan 31, 2019

What problem does this PR solve?

Format the log according to our RFC.

What is changed and how it works?

This PR is going to do the log format for tools, client, pkg, and table packages.
And it still works in progress. Do not review it now.

Check List

Tests

  • Unit test

@rleungx rleungx added type/enhancement The issue or PR belongs to an enhancement. do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. labels Jan 31, 2019
@codecov-io
Copy link

codecov-io commented Jan 31, 2019

Codecov Report

Merging #1424 into master will decrease coverage by 0.09%.
The diff coverage is 9.09%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #1424     +/-   ##
=========================================
- Coverage   67.74%   67.65%   -0.1%     
=========================================
  Files         158      158             
  Lines       15048    15076     +28     
=========================================
+ Hits        10194    10199      +5     
- Misses       3924     3951     +27     
+ Partials      930      926      -4
Impacted Files Coverage Δ
...ols/pd-simulator/simulator/cases/balance_leader.go 0% <0%> (ø) ⬆️
tools/pd-simulator/simulator/cases/add_nodes.go 0% <0%> (ø) ⬆️
tools/pd-simulator/simulator/event.go 0% <0%> (ø) ⬆️
pkg/metricutil/metricutil.go 90.62% <0%> (-9.38%) ⬇️
tools/pd-simulator/simulator/cases/hot_read.go 0% <0%> (ø) ⬆️
pkg/tempurl/tempurl.go 71.42% <0%> (+14.28%) ⬆️
tools/pd-simulator/simulator/drive.go 0% <0%> (ø) ⬆️
tools/pd-simulator/simulator/simutil/logger.go 0% <0%> (ø) ⬆️
tools/pd-simulator/simulator/cases/delete_nodes.go 0% <0%> (ø) ⬆️
.../pd-simulator/simulator/cases/add_nodes_dynamic.go 0% <0%> (ø) ⬆️
... and 23 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41f5261...0e45af7. Read the comment docs.

Signed-off-by: rleungx <rleungx@gmail.com>
Signed-off-by: rleungx <rleungx@gmail.com>
@rleungx rleungx removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Feb 15, 2019
@nolouch
Copy link
Contributor

nolouch commented Feb 18, 2019

/rebuild

@nolouch nolouch merged commit 376c933 into tikv:master Feb 18, 2019
@nolouch nolouch added the component/log Log. label Feb 18, 2019
nolouch pushed a commit to nolouch/pd that referenced this pull request Jul 10, 2019
* log format for tools, client, pkg, table, simulator

Signed-off-by: rleungx <rleungx@gmail.com>
nolouch pushed a commit to nolouch/pd that referenced this pull request Jul 10, 2019
* log format for tools, client, pkg, table, simulator

Signed-off-by: rleungx <rleungx@gmail.com>
nolouch pushed a commit to nolouch/pd that referenced this pull request Jul 10, 2019
* log format for tools, client, pkg, table, simulator

Signed-off-by: rleungx <rleungx@gmail.com>
youjiali1995 pushed a commit that referenced this pull request Jul 17, 2019
* *:Start swithing to the zap logger (#1422)

Signed-off-by: nolouch <nolouch@gmail.com>

* *: log format for schedule, scheduler, syncer, api (#1423)

* log format for schedule, scheduler, syncer, api

Signed-off-by: rleungx <rleungx@gmail.com>

* pdctl, api: support hex format keys (#1309)

pdctl, api: support hex format keys

Signed-off-by: disksing <i@disksing.com>

* *: log format for tools, client, pkg, table (#1424)

* log format for tools, client, pkg, table, simulator

Signed-off-by: rleungx <rleungx@gmail.com>

* *: log format for server (#1431)

* *: log format for server

Signed-off-by: nolouch <nolouch@gmail.com>

* *: log format for logutil, server/core (#1438)

* *: log format for logutil,server/core

* *: improve some logs (#1516)

* improve some logs

Signed-off-by: Ryan Leung <rleungx@gmail.com>

* log: do not add stack to error log (#1532)

* pkg, tools: improve the log message (#1636)

* pkg,tools: improve the log message

Signed-off-by: nolouch <nolouch@gmail.com>

* core: use hex format for region meta key (#1627)

Signed-off-by: Ryan Leung <rleungx@gmail.com>=

* address comments

Signed-off-by: nolouch <nolouch@gmail.com>

* address comments

Signed-off-by: nolouch <nolouch@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/log Log. type/enhancement The issue or PR belongs to an enhancement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants