Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Network sync refactoring (part 3) #11347

Conversation

nazar-pc
Copy link
Contributor

@nazar-pc nazar-pc commented May 4, 2022

This is the third step towards #8686.

As #8686 (comment) suggests, this moves light.v1 proto scheme and a bunch of data structures related to sync into new sc-network-light crate, which sc-network now depends on.

As in previous PR this is an implementation detail of sc-network for now.

A few minor changes, mostly just moving files around.

Polkadot address: 1vSxzbyz2cJREAuVWjhXUT1ds8vBzoxn2w4asNpusQKwjJd

@arkpar arkpar added the A0-please_review Pull request needs code review. label May 4, 2022
@arkpar arkpar requested a review from bkchr May 4, 2022 10:08
@nazar-pc
Copy link
Contributor Author

@bkchr friendly ping here before there are merge conflicts

@bkchr bkchr added B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels May 14, 2022
@bkchr
Copy link
Member

bkchr commented May 14, 2022

/tip medium

@substrate-tip-bot
Copy link

Please fix the following problems before calling the tip bot again:

  • Contributor did not properly post their Polkadot or Kusama address.

Make sure the pull request description has: "{network} address: {address}".

@bkchr
Copy link
Member

bkchr commented May 14, 2022

Ty @nazar-pc :) If you provide your polkadot address, I will call the bot again.

@bkchr
Copy link
Member

bkchr commented May 14, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Checks failed for a3ad69c

@nazar-pc
Copy link
Contributor Author

If you provide your polkadot address, I will call the bot again.

Done, thanks!

@bkchr bkchr added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label May 14, 2022
@bkchr
Copy link
Member

bkchr commented May 14, 2022

/tip medium

@substrate-tip-bot
Copy link

A medium tip was successfully submitted for nazar-pc (1vSxzbyz2cJREAuVWjhXUT1ds8vBzoxn2w4asNpusQKwjJd on polkadot).

https://polkadot.js.org/apps/?rpc=wss%3A%2F%2Frpc.polkadot.io#/treasury/tips

@bkchr
Copy link
Member

bkchr commented May 14, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 97a2bf9 into paritytech:master May 14, 2022
@nazar-pc nazar-pc deleted the network-sync-refactoring-part-3 branch May 14, 2022 07:18
godcodehunter pushed a commit to sensoriumxr/substrate that referenced this pull request Jun 22, 2022
* Move `light.v1.proto` schema into new crate `sc-network-light`

* Move `sc_network::light_client_requests` and submodule to `sc_network_light::light_client_requests`

* Fix apparently outdated reference in documentation and visibility modifier

* Fix rustdoc check

* Update lock file
DaviRain-Su pushed a commit to octopus-network/substrate that referenced this pull request Aug 23, 2022
* Move `light.v1.proto` schema into new crate `sc-network-light`

* Move `sc_network::light_client_requests` and submodule to `sc_network_light::light_client_requests`

* Fix apparently outdated reference in documentation and visibility modifier

* Fix rustdoc check

* Update lock file
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Move `light.v1.proto` schema into new crate `sc-network-light`

* Move `sc_network::light_client_requests` and submodule to `sc_network_light::light_client_requests`

* Fix apparently outdated reference in documentation and visibility modifier

* Fix rustdoc check

* Update lock file
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants