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

Companion PR for #7039: grandpa-rpc dont share subscription manager, only executor #1687

Merged
5 commits merged into from
Sep 14, 2020

Conversation

octol
Copy link
Contributor

@octol octol commented Sep 8, 2020

Companion PR for #7039.
Don't share subscription manager with grandpa-rpc, it is enough to only share the executor.

@octol octol added 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. labels Sep 8, 2020
Copy link
Contributor

@tomusdrw tomusdrw left a comment

Choose a reason for hiding this comment

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

lgtm!

rpc/src/lib.rs Outdated Show resolved Hide resolved
Copy link
Contributor

@andresilva andresilva left a comment

Choose a reason for hiding this comment

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

lgtm

node/service/src/lib.rs Outdated Show resolved Hide resolved
service/src/lib.rs Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Sep 14, 2020

Waiting for commit status.

@ghost
Copy link

ghost commented Sep 14, 2020

Head SHA changed; merge aborted.

@andresilva
Copy link
Contributor

bot merge

@ghost
Copy link

ghost commented Sep 14, 2020

Waiting for commit status.

@ghost ghost merged commit 9892aef into master Sep 14, 2020
@ghost ghost deleted the jon/grandpa-rpc-only-dep-on-executor branch September 14, 2020 11:30
ordian added a commit that referenced this pull request Sep 14, 2020
* master:
  Companion PR for #6984 (#1661)
  Update some dependencies. (#1718)
  Add a specific memory requirements (#1716)
  Companion PR for #7039: grandpa-rpc dont share subscription manager, only executor (#1687)
  Update bytes. (#1715)
  Add a note about memory requirements (#1714)
  Update parity-multiaddr. (#1700)
  typo in proxy tests (#1713)
  Companion PR for ` Add a `build-sync-spec` subcommand and remove the CHT roots from the light sync state.` (#1670)
  Forwardport: Validation: don't detect STDIN closing when running in process (#1695) (#1703)
This pull request was closed.
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants