-
Notifications
You must be signed in to change notification settings - Fork 648
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
Get block hashes by timestamp range #219
Conversation
|
||
Get Block Hashes by Timestamp range | ||
|
||
var time1 = 1441911000; // Notice time is in seconds not milliseconds |
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.
missing backticks for code formatting
8c6b991
to
1f2cfea
Compare
LGTM: tests pass locally |
If we can store the indexes as binary, e.g. sending a buffer for the key and value, and receiving a buffer the data storage size will be half the size, and faster to write and read. |
1f2cfea
to
3c128bb
Compare
Updated to binary encode leveldb key and value. |
3c128bb
to
78a65cc
Compare
78a65cc
to
00d3a0b
Compare
LGTM |
Testing with bitpay/insight-api#368 |
LGTM |
Get block hashes by timestamp range
Closes #204