Skip to content

Conversation

@marcvelmer
Copy link
Collaborator

Separated queues for both Dvote and Web3 Gateways and refactored randomFromDefault and randomfromUri code into the discovery class.

marcvelmer and others added 30 commits June 8, 2021 15:33
Changed order in WEB3 initial calls for bailing earlier
Changed getBlockNums for 'lazy' or 'on demand' usage, only called when required
Added custom errors
Changed GWs sort criteria
Added new TODOs for `initEns` and `isReady` functions
…stent with actual HTTP calls instead of Websocket

Added new `isPrepared()` function replacing `isReady()` internal usage
Added new TODO in `isPrepared()` for simplifying internal status calls
…l() of all metrics to be set in each Gateway, achieving a better response time because of doing the old `isUp` in a parallel `Promise.all()`

Added some TODOs
Changed `tsconfig.json` target `es2020` for 'es6', but needs to be reviewed
Added few conditions for discarding Web3 Gateways
Partly refactored GatewayDiscovery class
Added few TODOs
…single function that will be removed in the future
Changed DVote Gateways order criteria, now performance time is the preferred metric
Refactor some internal DVote Gateway functions and add some docs
Added some TODO related to custom error handling
…ility with TODO for removing in the future
Refactor and fixed timeout handling when filtering healthy nodes
Added few TODOs and fixed minor things
Moved to target ES2020 for using `Promise.allSettled()`
…hat call the new functions in the gateway discovery.
Refactor and fixed timeout handling when filtering healthy nodes
Added few TODOs and fixed minor things
Moved to target ES2020 for using `Promise.allSettled()`
…hat call the new functions in the gateway discovery.
…gatewaypool_queue

# Conflicts:
#	src/net/gateway-discovery.ts
#	src/net/gateway-dvote.ts
#	src/net/gateway-web3.ts
#	src/net/gateway.ts
#	test/helpers/web3-service.ts
#	test/integration/block-numbers.ts
@marcvelmer marcvelmer requested review from brickpop and emmdim July 5, 2021 16:40
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.

3 participants