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

Rename Transactions back to Operations #822

Closed
decanus opened this issue Mar 21, 2019 · 5 comments
Closed

Rename Transactions back to Operations #822

decanus opened this issue Mar 21, 2019 · 5 comments
Labels
general:presentation Presentation (as opposed to content)

Comments

@decanus
Copy link
Contributor

decanus commented Mar 21, 2019

A few of us implementers have been talking about the naming of Transactions and believe it is best renamed back to Operations to lower confusion and potentially mistaking Transactions with transactions in the classical sense. The only thing that should be known as a Transaction is a Transfer.

If not, it would be great to know what the reason behind the rename was.

@GregTheGreek
Copy link
Contributor

+1

@djrtwo
Copy link
Contributor

djrtwo commented Mar 21, 2019

I'm pro :)
Spent a long time coming up with the name Operations and was sad to see it go

@JustinDrake
Copy link
Collaborator

JustinDrake commented Mar 21, 2019

it would be great to know what the reason behind the rename was

Abstractly, a "blockchain" is a chain of "blocks" that advance "state", where the blocks contain signed messages called "transactions". The beacon chain is a "system blockchain" which has "system blocks", "system state" and... "system transactions".

In summary, the word "transaction" was picked to fit the block/state/transaction framework. While I think the nomenclature is natural I can see how it can be confusing. Looks like people are in favour of renaming back to "operation" :)

@decanus
Copy link
Contributor Author

decanus commented Mar 21, 2019

@JustinDrake thanks for the full clarification. As we have user transactions on the beaconchain now I’d definitely support going back to operations.

@JustinDrake
Copy link
Collaborator

As we have user transactions on the beaconchain now I’d definitely support going back to operations.

Sure. (As a side note, transfers are a temporary thing to be removed in phase 2.)

@hwwhww hwwhww added the general:presentation Presentation (as opposed to content) label Mar 22, 2019
JustinDrake added a commit that referenced this issue Mar 31, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
general:presentation Presentation (as opposed to content)
Projects
None yet
Development

No branches or pull requests

5 participants