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

Spec out transaction ingestion logic within the context of the OVM #349

Closed
karlfloersch opened this issue Jul 22, 2019 · 0 comments
Closed
Labels
@pigi/aggregator Tasks for @pigi/aggregator

Comments

@karlfloersch
Copy link
Contributor

Is your feature request related to a problem? Please describe.
In our previous aggregator design the transaction ingestion logic looked a lot like the StateManager logic. However, with our recent OVM architecture this becomes less clear because it's harder to "fudge" the StateManger in this way.

In particular the way we were modifying the StateManager is before we were assuming inclusion of whatever StateUpdate that executeTransaction(...) produced. Now, because the OVM will automate guarding if we do it correctly, it's less clear how to do this.

Describe the solution you'd like
There are two main approaches for how to solve this problem:

  1. Aggregator ingests transactions as proofs that are applied to the OVM, and then inspects decisions made on the OVM to determine if that transaction should be applied in the next block.
  2. Aggregator runs custom logic which is special built for handling transaction authentication that in it's simplest form is a rangeDB with all of the currently authenticated owners of a particular range. If a message + signature is received by an owner the message is queued for the next block.

The 2nd solution is much less general purpose, but it's quite straightforward to implement especially under the assumption that only one user is able to queue up a new SU to be included. In the long term transaction inclusion will have to take into consideration things like transaction fees, which will further complicate things. The unknown unknowns are probably the best argument for approach (2).

@karlfloersch karlfloersch added enhancement New feature or request @pigi/aggregator Tasks for @pigi/aggregator question Need more information and removed enhancement New feature or request question Need more information labels Jul 22, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@pigi/aggregator Tasks for @pigi/aggregator
Projects
None yet
Development

No branches or pull requests

1 participant