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

dmp-queue: Store messages if already processed more than the maximum #2343

Merged
merged 2 commits into from
Mar 17, 2023

Conversation

bkchr
Copy link
Member

@bkchr bkchr commented Mar 17, 2023

No description provided.

@bkchr bkchr added B0-silent Changes should not be mentioned in any release notes A0-please_review Pull request needs code review. C3-medium PR touches the given topic and has a medium impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited labels Mar 17, 2023
@bkchr bkchr requested a review from gavofyork March 17, 2023 10:17
@@ -189,6 +189,8 @@ pub mod pallet {
},
/// Downward message from the overweight queue was executed.
OverweightServiced { overweight_index: OverweightIndex, weight_used: Weight },
/// The maximum number of downward messages was.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
/// The maximum number of downward messages was.
/// The maximum number of downward messages was exhausted and the message enqueued for
/// later processing.

@gavofyork
Copy link
Member

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 1782170 into master Mar 17, 2023
@paritytech-processbot paritytech-processbot bot deleted the bkchr-dmp-queue branch March 17, 2023 13:45
bkchr added a commit that referenced this pull request Mar 17, 2023
…2343)

* dmp-queue: Store messages if already processed more than the maximum

* Put new event at the end
bkchr added a commit that referenced this pull request Mar 17, 2023
…2343)

* dmp-queue: Store messages if already processed more than the maximum

* Put new event at the end
EgorPopelyaev pushed a commit that referenced this pull request Mar 20, 2023
…2343)

* dmp-queue: Store messages if already processed more than the maximum

* Put new event at the end
joepetrowski pushed a commit that referenced this pull request Mar 20, 2023
…2343) (#2347)

* dmp-queue: Store messages if already processed more than the maximum

* Put new event at the end

Co-authored-by: Bastian Köcher <[email protected]>
EgorPopelyaev pushed a commit that referenced this pull request Mar 20, 2023
…2343)

* dmp-queue: Store messages if already processed more than the maximum

* Put new event at the end
ordian added a commit that referenced this pull request Mar 21, 2023
* master:
  Companion for #13624 (#2354)
  Introduce Fellowship into Collectives (#2186)
  NFTs 2.0 on Statemine (#2314)
  Bump assert_cmd from 2.0.8 to 2.0.10 (#2341)
  Bump clap from 4.1.8 to 4.1.11 (#2352)
  Companion for substrate #13312: Rename `Deterministic` to `Enforce` (#2350)
  [Companion #13634] keystore overhaul (iter) (#2345)
  Revert #2304 (#2349)
  Deprecate Currency: Companion for #12951 (#2334)
  Bump ci-linux image for rust 1.68
  Always pass port to jsonrpsee WebSocket client (#2339)
  bump zombienet to v1.3.40 (#2348)
  Improve build times by disabling wasm-builder in `no_std` (#2308)
  Bump toml from 0.7.2 to 0.7.3 (#2340)
  Bump serde from 1.0.152 to 1.0.156 (#2329)
  Parachains should charge for proof size weight (#2326)
  dmp-queue: Store messages if already processed more than the maximum (#2343)
  [Companion  #13615] Keystore overhaul (#2336)
  Bump quote from 1.0.23 to 1.0.26 (#2331)
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 C3-medium PR touches the given topic and has a medium impact on builders. D9-needsaudit 👮 PR contains changes to fund-managing logic that should be properly reviewed and externally audited
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants