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

Bump polkadot deps and reactivate light client #340

Closed
wants to merge 1 commit into from

Conversation

wirednkod
Copy link

I tried bumping the polkadot to latest deps (9.7.1) that contain the fix of smoldot light client - concerning the returns of state_getKeysPaged.
Unfortunately the return time (~45 seconds) are not satisfactory for the Dashboard which results to:

logger.js:62 2022-11-10 11:04:32        RPC-CORE: getKeysPaged(key: StorageKey, count: u32, startKey?: StorageKey, at?: BlockHash): Vec<StorageKey>:: -32000: Storage query errors:
- Timeout
11:04:32.530 logger.js:62 2022-11-10 11:04:32        RPC-CORE: getKeysPaged(key: StorageKey, count: u32, startKey?: StorageKey, at?: BlockHash): Vec<StorageKey>:: -32000: Storage query errors:
- ConnectionShutdown
11:04:32.706 logger.js:62 2022-11-10 11:04:32        RPC-CORE: getKeysPaged(key: StorageKey, count: u32, startKey?: StorageKey, at?: BlockHash): Vec<StorageKey>:: -32000: No node available for storage query

I though do not think that Staking Dashboard should exclude Light Client support, just because some info cannot return (on time or not supported still).

This PR bumps the deps of polkadot/api and in addition, reactivates the light client with the experimental flag - but hides the parts that are not returned correctly from light client at the moment. I think that these should be reported to Smoldot Repo, and be tested in new releases.

@rossbulat
Copy link
Collaborator

Dependencies are now updated on master.

@rossbulat rossbulat closed this Nov 12, 2022
@rossbulat rossbulat deleted the nik-bump-polkadot-api branch December 11, 2022 02:03
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