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

fix(chain-connector): parse chain errors, and also other stuff [fixes NET-646, NET-698, NET-690] #162

Merged
merged 7 commits into from
Dec 28, 2023

Conversation

kmd-fl
Copy link
Collaborator

@kmd-fl kmd-fl commented Dec 21, 2023

  • Parse chain errors
    This helps a lot in situations when the chain RPC returns errors for some deals. Previously, the whole response wouldn't be parsed, and proper results for some deals would be ignored. Now, the deal with okay responses will be processed. This change works for deal updates, deal removal events, and deal statuses.
  • Increase range block to 2000. Look in the NET-646 task for details
  • Sort responses of batch requests by JSON-RPC ids so that the response matches its deal since the JSON-RPC doesn't guarantee the order preservance.
  • Fixed worker triggering on worker update

@kmd-fl kmd-fl requested review from folex and justprosh December 21, 2023 16:47
@folex folex added the e2e label Dec 27, 2023
@folex folex enabled auto-merge (squash) December 28, 2023 21:06
@folex folex merged commit 677574c into main Dec 28, 2023
9 checks passed
@folex folex deleted the better-connector branch December 28, 2023 22:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants