Skip to content

Comments

Mempool - Sort txns#21

Open
bottlebo wants to merge 3 commits intoSilentNotaryEcosystem:masterfrom
bottlebo:Mempool-TxSort
Open

Mempool - Sort txns#21
bottlebo wants to merge 3 commits intoSilentNotaryEcosystem:masterfrom
bottlebo:Mempool-TxSort

Conversation

@bottlebo
Copy link
Contributor

@bottlebo bottlebo commented Apr 18, 2019

Идея в том, что при разматывании блока в mempool (при конфликте, к примеру) в новый блок они должны попасть перед теми, которые на них ссылаются.

@trueshura
Copy link
Collaborator

Все вроде ничего, но:

Очевидно что это не добавит скорости ноде. А учитывая, что вся эта сортировка нужна только на нодах свидетеля (сделай поиск по getFinalTxns), то я бы сортировал не в момент прихода транзакции, а в момент getFinalTxns. Это:

  • сразу избавит рядовые ноды от этой ненужной им нагрузки.
  • выбираться будут только те транзакции, что принадлежат groupId (что еще снизит выборку).
  • позволит сразу освободить память DAG (в твоей реализации он будет разрастаться чудовищно!)

@trueshura
Copy link
Collaborator

Ну и назвал бы я функцию не sortTxns, а order
Мелочь, но понятнее, как мне кажется.

@trueshura trueshura self-requested a review May 2, 2019 10:07
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.

2 participants