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

Adjust max nominators down to 128 (from 256) #1782

Merged
merged 1 commit into from
Oct 5, 2020
Merged

Conversation

jacogr
Copy link
Contributor

@jacogr jacogr commented Oct 2, 2020

No description provided.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Oct 2, 2020
@s3krit s3krit added the C1-low PR touches the given topic and has a low impact on builders. label Oct 2, 2020
@Swader
Copy link

Swader commented Oct 3, 2020

Is there a link to a discussion on reasoning behind this?

@kianenigma
Copy link
Contributor

Is there a link to a discussion on reasoning behind this?

Nothing external, we decided that the initial move to 256 was maybe a bit too much. Rest assured, 128 is more sensible now as no polkadot validator will become oversubscribed with it. Over time, the governance can and should adjust this value based on the network status.

@gavofyork gavofyork merged commit 27c3b75 into master Oct 5, 2020
@gavofyork gavofyork deleted the jg-max-noms branch October 5, 2020 14:02
ordian added a commit that referenced this pull request Oct 6, 2020
* master:
  Make collation an optional return (#1787)
  XCM: Land xcm-handler and xcm-executor (#1771)
  v0.8.25 (#1785)
  add two node local net script (#1781)
  Adjust max nominators down to 128 (from 256) (#1782)
  Companion for substrate/pull/7215 (#1768)
  Remove Stale Upgrades (#1780)
  Update Polkadot Weights for Substrate 2.0 (#1761)
  Parachains v1 registrar module. (#1559)
  Derive `From` for `AllMessages` and simplify `send_msg` (#1774)
  implement remaining subsystem metrics (#1770)
  Companion for paritytech/substrate#7236 (#1773)
  WIP: remove deprecated only/except clauses, build is now manual on PRs (#1769)
  Increase Westend `spec_version` (#1766)
  move Metrics to utils (#1765)
ordian added a commit that referenced this pull request Oct 6, 2020
* master:
  NetworkBridge: validator (authorities) discovery api (#1699)
  Registrar v1 follow-ups (#1786)
  Make collation an optional return (#1787)
  XCM: Land xcm-handler and xcm-executor (#1771)
  v0.8.25 (#1785)
  add two node local net script (#1781)
  Adjust max nominators down to 128 (from 256) (#1782)
  Companion for substrate/pull/7215 (#1768)
  Remove Stale Upgrades (#1780)
  Update Polkadot Weights for Substrate 2.0 (#1761)
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. 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.

7 participants