-
Notifications
You must be signed in to change notification settings - Fork 18
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
Revise near_indexer::StreamerMessage
struct
#1
Comments
https://hackmd.io/@uYOqIgyuTWytaOXsS1rgxA/HkAeeQBCF @telezhnaya @frol please, have a look at the |
This PR on nearcore side near/nearcore#6255 should resolve our basic concerns about |
Current
|
@frol a bit fresher one
|
@frol are we waiting for the merging of the PR on the nearcore side before closing this issue or we want to perform some action points? |
@khorolets I suggest we also drop the duplicate |
During the 1-on-1 we've decided to leave @frol should we close this issue? |
Yes, we decided that this is not a huge deal given that I was only talking about the framework consumer side of the StreamerMessage (I did not suggest to cut the chunk headers from the block.json file on S3) |
We assume
StreamerMessage
contains duplicates we want to avoid. It should revise the struct to find out what can be optimized there or dropped from.The text was updated successfully, but these errors were encountered: