-
Notifications
You must be signed in to change notification settings - Fork 11
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
Having the wrong start block generates non-intuitive error messages #27
Comments
Note also that the firehose grpc service starts fine with the wrong start block. Probably it should do the same check as substreams? |
That "check" at the launch of substreams was a bad idea. We now have the flag With this (either hardcoded in It doesn't solve your need to automatically detect a config error (which will manifest later when trying to get blocks) but it will probably bubble up as errors that make more sense (like "no linkable block" or something like that). A runtime check of the existing block is IMHO a bad idea, it causes quite a few edge cases when bootstrapping chains or starting up with a chain that doesn't have the first 100 merged blocks file yet |
Just to note, Johnathan from my team ran into this problem again recently. |
See also: #62 |
firehose-antelope is hard-coded to start at block 2.
In this chain, the first start block is actually 3.
After upgrading to latest firehose-core, the substreams tier1 node doesn't start. Log is below. Settting
common-first-streamable-block: 3
allows the substreams tier1 node to startIn earlier versions of the software I guess having the correct start block was not checked.
Perhaps the error message could be more clear?
Maybe print a message it is checking the start block can be found? Not sure what is best. It took a bit to figure out the problem from the debug messages available.
The text was updated successfully, but these errors were encountered: