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

CPS-0018? | Greater Transaction Throughput #925

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

bwbush
Copy link

@bwbush bwbush commented Oct 11, 2024

The Cardano mainnet occasionally experiences congestion where there are too many transactions in the memory pool to be included in the next block or in the next few blocks. Sometimes the block utilization peaks above 90% for an extended period of time. This not only impacts general user experience but it can also severely impact use cases such as airdrops, oracles, partner chains, DEXes, and Dapps. Emerging use cases and application deployments promise to accelerate the need for high throughput on Cardano. Applied research on several fronts is needed to propose and provide evidence for techniques that increase throughput measured in terms of transactions, transaction size, and script execution units. Such work should be based on a clear understanding of stakeholder requirements.


(rendered latest document)

@rphair rphair changed the title CPS for greater transaction throughput CPS-???? | Greater Transaction Throughput Oct 12, 2024
@rphair rphair added State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. Category: Consensus Proposals belonging to the `Consensus` category. and removed State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. labels Oct 12, 2024
@rphair
Copy link
Collaborator

rphair commented Oct 12, 2024

thanks @bwbush & @abailly-iohk - I'd tagged this submission Triage before noticing it was still a draft. If you would like some feedback at the upcoming CIP meeting (https://hackmd.io/@cip-editors/98) please say so and I'll apply that tag again.

Also please update the "latest document" link above if & when the added pathname to your README.md changes.

- Removed extraneous notes.
- Redrew three plots.
- Added stronger justification.
- Fixed link to CPS-17.
@bwbush bwbush marked this pull request as ready for review November 13, 2024 20:41
@bwbush
Copy link
Author

bwbush commented Nov 13, 2024

@rphair, I made a few revisions in 97908bd and marked this ready for review.

@rphair rphair added the State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. label Nov 14, 2024
@rphair
Copy link
Collaborator

rphair commented Nov 14, 2024

hajola and others added 2 commits November 21, 2024 16:49
License: Apache-2.0
---

# CIP-???? Greater Transaction Throughput
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
# CIP-???? Greater Transaction Throughput

Title not included as header as part of template


The Cardano mainnet occasionally experiences congestion where there are too many transactions in the memory pool to be included in the next block or in the next few blocks. Sometimes the block utilization peaks above 90% for an extended period of time. This not only impacts general user experience but it can also severely impact use cases such as airdrops, oracles, partner chains, DEXes, and Dapps. Emerging use cases and application deployments promise to accelerate the need for high throughput on Cardano. Applied research on several fronts is needed to propose and provide evidence for techniques that increase throughput measured in terms of transactions, transaction size, and script execution units. Such work should be based on a clear understanding of stakeholder requirements.

## Motivation
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
## Motivation

please tweak the structure to align with the template
https://github.com/cardano-foundation/CIPs/blob/master/.github/CPS-TEMPLATE.md

Copy link
Collaborator

Choose a reason for hiding this comment

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

@bwbush it could also be easier to re-work by following the content outline indicated in https://github.com/cardano-foundation/CIPs/tree/master/CIP-9999 - especially in the case that this CPS might have been written as a CIP originally (to help sift the content of this draft between the CPS & its associate CIP). Then the required outline will also conform to the template as a matter of course.

Copy link
Collaborator

@rphair rphair left a comment

Choose a reason for hiding this comment

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

@bwbush this whole series of Consensus submissions has been well received so far & both CPSs and their corresponding CIPs should all have candidate numbers assigned by now. Eventually we would like each CIP to link to the CPS that it "solves" (and vice versa, including in the metadata header). I believe it's a benefit to the community that each solution comes with its own definition of the problem space, to potentially include improvements offered by others.

Please also assign this number by renaming the containing folder & linking to the new pathname of your README file in the original posting at the beginning of the PR. 🎉

@@ -0,0 +1,137 @@
---
CPS: 0THR
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
CPS: 0THR
CPS: 18

@rphair rphair added State: Confirmed Candiate with CIP number (new PR) or update under review. and removed State: Triage Applied to new PR afer editor cleanup on GitHub, pending CIP meeting introduction. labels Nov 26, 2024
@rphair rphair changed the title CPS-???? | Greater Transaction Throughput CPS-0018? | Greater Transaction Throughput Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Consensus Proposals belonging to the `Consensus` category. State: Confirmed Candiate with CIP number (new PR) or update under review.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants