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

Public mempool API to query the size limit #1510

Closed
deepfire opened this issue Jan 27, 2020 · 1 comment · Fixed by #1509
Closed

Public mempool API to query the size limit #1510

deepfire opened this issue Jan 27, 2020 · 1 comment · Fixed by #1509
Assignees
Labels
consensus issues related to ouroboros-consensus mempool
Milestone

Comments

@deepfire
Copy link
Contributor

End users would benefit from obtaining information regarding the limit of the mempool, such that, for example, the MempoolSize (and, hence MempoolSnapshot) could be extended to contain not only the its current byte/Tx occupancy, but also the byte limit:

-- | The size of a mempool.
data MempoolSize = MempoolSize
  { msNumTxs   :: !Word32
    -- ^ The number of transactions in the mempool.
  , msNumBytes :: !Word32
    -- ^ The summed byte size of all the transactions in the mempool.
  , msMaxBytes :: !Word32
    -- ^ The upper limit on the byte size of the mempool.
  } deriving (Eq, Show)
@deepfire deepfire added the consensus issues related to ouroboros-consensus label Jan 27, 2020
@deepfire
Copy link
Contributor Author

deepfire commented Jan 27, 2020

implementation: #1509

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
consensus issues related to ouroboros-consensus mempool
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants