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

*: move "snap" to "internal/raftsnap" #9211

Merged
merged 7 commits into from
Jan 24, 2018
Merged

*: move "snap" to "internal/raftsnap" #9211

merged 7 commits into from
Jan 24, 2018

Conversation

gyuho
Copy link
Contributor

@gyuho gyuho commented Jan 23, 2018

Fix #9207.
For #9220.

@codecov-io
Copy link

Codecov Report

Merging #9211 into master will decrease coverage by <.01%.
The diff coverage is 79.16%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #9211      +/-   ##
==========================================
- Coverage   76.06%   76.05%   -0.01%     
==========================================
  Files         363      363              
  Lines       30161    30161              
==========================================
- Hits        22941    22938       -3     
- Misses       5625     5631       +6     
+ Partials     1595     1592       -3
Impacted Files Coverage Δ
raftsnap/db.go 63.33% <ø> (ø)
raftsnap/metrics.go 100% <ø> (ø)
raftsnap/snapshotter.go 72.04% <ø> (ø)
raftsnap/message.go 100% <ø> (ø)
etcdserver/storage.go 57.89% <100%> (ø) ⬆️
etcdserver/snapshot_merge.go 80% <100%> (ø) ⬆️
rafthttp/http.go 79.62% <100%> (ø) ⬆️
snapshot/v3_snapshot.go 62.96% <100%> (ø) ⬆️
rafthttp/snapshot_sender.go 91.04% <100%> (ø) ⬆️
rafthttp/peer.go 91.72% <100%> (ø) ⬆️
... and 27 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 8025082...a8c3b3d. Read the comment docs.

@xiang90
Copy link
Contributor

xiang90 commented Jan 24, 2018

lgtm. maybe we also need to add doc.go into raftsnap pkg to clarify that it is used by etcd internally for raft snapshot. (or it might be the time for use to move all internal pkg to /internal :P)

@gyuho
Copy link
Contributor Author

gyuho commented Jan 24, 2018

@xiang90 Added doc.go and moved to internal with a separate issue to track other packages #9220.

@gyuho gyuho merged commit 7331949 into etcd-io:master Jan 24, 2018
@gyuho gyuho deleted the raftsnap branch January 24, 2018 19:19
@gyuho gyuho changed the title *: rename "snap" to "raftsnap" *: rename "snap" to "internal/raftsnap" Jan 25, 2018
@gyuho gyuho changed the title *: rename "snap" to "internal/raftsnap" *: move "snap" to "internal/raftsnap" Jan 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants