Skip to content
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

Kan/v0.11 access older results #131

Closed
wants to merge 7 commits into from
Closed

Conversation

Kay-Zee
Copy link
Member

@Kay-Zee Kay-Zee commented Nov 9, 2020

This adds the ability to talk to historical access nodes.

Will only call the previous access node if the Tx cannot be found on the current node.

if status.Code(err) == codes.NotFound {
continue
}
// TODO should we do something if the error isn't not found?
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

log that the error was not "NotFound"

@Kay-Zee Kay-Zee changed the base branch from v0.11 to master November 25, 2020 18:35
@Kay-Zee Kay-Zee changed the base branch from master to v0.11 November 25, 2020 18:35
@Kay-Zee
Copy link
Member Author

Kay-Zee commented Nov 25, 2020

closing in favour of #181

@Kay-Zee Kay-Zee closed this Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants