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

XLS-56d: Atomic/Batch Transactions #197

Merged
merged 21 commits into from
Jan 23, 2025
Merged

XLS-56d: Atomic/Batch Transactions #197

merged 21 commits into from
Jan 23, 2025

Conversation

mvadari
Copy link
Collaborator

@mvadari mvadari commented May 6, 2024

Discussion thread can be found here: #162

This PR will be left open for easier commenting of specific lines/sections. The discussion will also still be available for ease of commenting for non-technical people.

@mvadari mvadari marked this pull request as draft May 6, 2024 17:07
Copy link
Collaborator

@mDuo13 mDuo13 left a comment

Choose a reason for hiding this comment

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

I am somewhat concerned about the greater potential for front-running with batch transactions. It seems like that is the "use case" that receives the biggest direct benefit from this feature even though we would rather discourage front-running.

Related, I think the specification should make it explicit how the batch affects the canonical order of transactions. It seems like the intent is for the canonical order to contain the outer transaction followed by the successful inner transactions in direct sequence, and that would start from wherever the outer transaction gets placed in the canonical order?

It might also be worth adding a subsection to Security about transaction fees. Specifically, make a note about how the outer transaction pays fees for all the inner transactions even if using modes like ONLYONE where some of those transactions can fail, and how this is a precaution against exploits that could make the whole network do the work of processing candidate transactions that won't pay transaction fees.

XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Show resolved Hide resolved
@mvadari mvadari marked this pull request as ready for review August 19, 2024 13:44
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
@dangell7

This comment was marked as outdated.

@dangell7

This comment was marked as outdated.

@dangell7

This comment was marked as outdated.

@mvadari

This comment was marked as resolved.

@shawnxie999

This comment was marked as resolved.

@mvadari

This comment was marked as outdated.

XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
Copy link

@ckeshava ckeshava left a comment

Choose a reason for hiding this comment

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

What's the influence of Batch transactions on the open-ledger cost? For instance, if 10 Batch transactions are included in a validated ledger, this is equivalent to inserting at most 80 non-Batch "usual" transactions. Is the soft-limit of the open-ledger cost calculation proportionately increased by this amendment?

@mvadari mvadari requested a review from dangell7 January 13, 2025 16:52
@mvadari
Copy link
Collaborator Author

mvadari commented Jan 13, 2025

What's the influence of Batch transactions on the open-ledger cost? For instance, if 10 Batch transactions are included in a validated ledger, this is equivalent to inserting at most 80 non-Batch "usual" transactions. Is the soft-limit of the open-ledger cost calculation proportionately increased by this amendment?

The open-ledger cost is tied to the total fee of all the Batch transactions, on the outer transaction.

mDuo13

This comment was marked as resolved.

@mvadari mvadari requested a review from mDuo13 January 13, 2025 22:06
XLS-0056d-batch/README.md Outdated Show resolved Hide resolved
XLS-0056d-batch/README.md Show resolved Hide resolved
@mvadari mvadari merged commit 497d86c into XRPLF:master Jan 23, 2025
@mvadari mvadari deleted the batch branch January 23, 2025 21:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants