-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Remove useBridgeReader and useSpanProducerReader flags from Bor API
#16433
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
Conversation
useBridgeReader and useSpanProducerReader flags Bor API
useBridgeReader and useSpanProducerReader flags Bor APIuseBridgeReader and useSpanProducerReader flags from Bor API
| if base.useBridgeReader { | ||
| logger.Info("starting rpc with polygon bridge") | ||
| } | ||
| logger.Info("starting rpc with polygon bridge") |
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.
This was added for debugging and user information during the transition to Astrid, I think it makes sense to remove it now.
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.
I will remove it in another PR so that I don't retrigger the CI checks.
cleanup follow up after #16433
…PI (#16433) Co-authored-by: antonis19 <[email protected]>
cherry-pick to release/3.1 of the following PRs: #16404 #16409 #16411 #16433 #16458 #16464 #16520 #16556 --------- Co-authored-by: antonis19 <[email protected]> Co-authored-by: Mark Holt <[email protected]>
No description provided.