-
Notifications
You must be signed in to change notification settings - Fork 997
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
Persist ETH1 Information #4305
Persist ETH1 Information #4305
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4305 +/- ##
========================================
Coverage ? 49.6%
========================================
Files ? 180
Lines ? 12731
Branches ? 0
========================================
Hits ? 6315
Misses ? 5427
Partials ? 989 |
message LatestETH1Data { | ||
uint64 block_height = 2; | ||
uint64 block_time = 3; | ||
repeated bytes block_hash = 4; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we need ssz-size tag for this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we do not need it, we are just using protobufs to serialize this for disk storage.
Can you please add the purpose of the Eth1ChainData message in the description? |
Added in a comment on it @terencechain |
beacon-chain/db/kv/powchain.go
Outdated
@@ -0,0 +1 @@ | |||
package kv |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's up with this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 on this package name, it can be specified in the .proto file
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
removed it
* add data structures * generate proto * add in new fields * Merge branch 'master' of https://github.com/prysmaticlabs/geth-sharding into saveETH1Data * add comments * Merge branch 'master' into saveETH1Data * remove file * Merge branch 'saveETH1Data' of https://github.com/prysmaticlabs/geth-sharding into saveETH1Data * Merge branch 'master' into saveETH1Data * Merge branch 'master' into saveETH1Data * Merge refs/heads/master into saveETH1Data * Merge refs/heads/master into saveETH1Data
* add data structures * generate proto * add in new fields * Merge branch 'master' of https://github.com/prysmaticlabs/geth-sharding into saveETH1Data * add comments * Merge branch 'master' into saveETH1Data * remove file * Merge branch 'saveETH1Data' of https://github.com/prysmaticlabs/geth-sharding into saveETH1Data * Merge branch 'master' into saveETH1Data * Merge branch 'master' into saveETH1Data * Merge refs/heads/master into saveETH1Data * Merge refs/heads/master into saveETH1Data
Part of #1430
Eth1ChainData
is used represent all our eth1 chain data so that it can be serialized for persistent storage.