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

Fix new CF missing BlobFileSizeCollector #78

Merged
merged 7 commits into from
Sep 18, 2019

Conversation

yiwu-arbug
Copy link
Collaborator

Summary:
On CreateColumnFamilies we should set BlobFileSizeCollectorFactory. Fixing it.

Test Plan:
added test

Signed-off-by: Yi Wu yiwu@pingcap.com

Signed-off-by: Yi Wu <yiwu@pingcap.com>
Copy link
Member

@Connor1996 Connor1996 left a comment

Choose a reason for hiding this comment

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

LGTM

Yi Wu added 3 commits September 18, 2019 00:47
Signed-off-by: Yi Wu <yiwu@pingcap.com>
Signed-off-by: Yi Wu <yiwu@pingcap.com>
@yiwu-arbug
Copy link
Collaborator Author

seems CI finds a data race case. will fix.

@codecov-io
Copy link

Codecov Report

Merging #78 into master will increase coverage by 0.33%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master      #78      +/-   ##
==========================================
+ Coverage   81.71%   82.04%   +0.33%     
==========================================
  Files          44       44              
  Lines        3122     3124       +2     
==========================================
+ Hits         2551     2563      +12     
+ Misses        571      561      -10

@yiwu-arbug yiwu-arbug merged commit fc4106d into tikv:master Sep 18, 2019
@yiwu-arbug yiwu-arbug deleted the stress_collector branch September 18, 2019 20:57
yiwu-arbug pushed a commit to yiwu-arbug/titan that referenced this pull request Sep 24, 2019
Summary:
On `CreateColumnFamilies` we should set `BlobFileSizeCollectorFactory`. Fixing it.

Test Plan:
added test

Signed-off-by: Yi Wu <yiwu@pingcap.com>
Connor1996 pushed a commit that referenced this pull request Sep 24, 2019
cherry-pick following patches
```
ff30897 2019-09-23 zbk602423539@gmail.. reset gc mark after sampling (#88)
cb67cd5 2019-09-22 zbk602423539@gmail.. fix wrong property after restart (#82)
c6e408c 2019-09-20 yiwu@pingcap.com     Fix blob file format description (#83)
fc4106d 2019-09-18 yiwu@pingcap.com     Fix new CF missing BlobFileSizeCollector (#78)
10710bb 2019-09-18 zbk602423539@gmail.. Add more metrics (#79)
8768067 2019-09-17 yiwu@pingcap.com     Prevent CF being dropped while GC is running (#72)
07aa065 2019-09-16 zbk602423539@gmail.. Rename version_set to blob_file_set (#69)
f963b88 2019-09-16 zbk602423539@gmail.. support histogram for titan stats (#74)
144e20f 2019-09-12 wujy.cs@gmail.com    gc breakdown metrics (#73)
715dbd6 2019-09-11 zbk602423539@gmail.. Update BlobFileMeta format (#68)
468ddc9 2019-09-03 yiwu@pingcap.com     Add internal operation stats and dump to info log periodically (#62)
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants