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

perf: optional SMT trie feature and migrate command #859

Merged
merged 7 commits into from
Dec 1, 2022

Conversation

blckngm
Copy link
Contributor

@blckngm blckngm commented Nov 14, 2022

SMT has recently got the trie feature, which will greatly decrease the amount of store read/write operations needed to update SMTs, thus improving transaction throughput and block syncing speed. SMT roots and proofs won't change.

This feature can be enabled via the smt-trie feature in godwoken crates, which is not enabled by default for now. Db with existing data needs to be migrated to use this new feature. See the new migrate command in godwoken-bin.

@gw-bot
Copy link

gw-bot bot commented Nov 14, 2022

Running integration test

Workflow Run Id: 3463071902

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

failure

@gw-bot
Copy link

gw-bot bot commented Nov 18, 2022

Running integration test

Workflow Run Id: 3495549850

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@gw-bot
Copy link

gw-bot bot commented Nov 21, 2022

Running integration test

Workflow Run Id: 3512967308

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@gw-bot
Copy link

gw-bot bot commented Nov 22, 2022

Running integration test

Workflow Run Id: 3519904026

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

failure

@gw-bot
Copy link

gw-bot bot commented Nov 23, 2022

Running integration test

Workflow Run Id: 3530282730

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@gw-bot
Copy link

gw-bot bot commented Nov 25, 2022

Running integration test

Workflow Run Id: 3545651799

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

failure

@gw-bot
Copy link

gw-bot bot commented Nov 26, 2022

Running integration test

Workflow Run Id: 3553178126

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@gw-bot
Copy link

gw-bot bot commented Nov 28, 2022

Running integration test

Workflow Run Id: 3561470848

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@blckngm blckngm changed the title perf: use SMT trie feature perf!: optional SMT trie feature Nov 28, 2022
@blckngm blckngm marked this pull request as ready for review November 28, 2022 03:42
@gw-bot
Copy link

gw-bot bot commented Nov 28, 2022

Running integration test

Workflow Run Id: 3563067412

Components:

Manually running integration test

Post a comment contains

/itest
[prebuilds: tag]
[godwoken: branch/ref]
[scripts: branch/ref]
[polyjuice: branch/ref]
[web3: branch/ref]
[kicker: branch/ref]
[tests: branch/ref]

Note: [] means optional, for example

/itest
prebuilds: dev-202203280240
godwoken: develop
scripts: 81676d9d53ffdf5bbaa60483928d07da16eb4a88
polyjuice: e37553b9

Run Result

success

@Flouse Flouse requested review from jjyr, zeroqn and magicalne and removed request for jjyr and zeroqn November 29, 2022 16:50
@Flouse Flouse added the enhancement New feature or request label Nov 29, 2022
@blckngm blckngm changed the title perf!: optional SMT trie feature perf: optional SMT trie feature and migrate command Nov 30, 2022
jjyr
jjyr previously approved these changes Nov 30, 2022
@jjyr jjyr merged commit a273f57 into godwokenrises:develop Dec 1, 2022
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

## Reference
- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
Flouse added a commit to Flouse/godwoken that referenced this pull request Feb 11, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see godwokenrises#859
jjyr pushed a commit that referenced this pull request Feb 13, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see #859
jjyr pushed a commit that referenced this pull request Feb 13, 2023
- add feature-tag at the end of image tag

- optional SMT trie feature
  see #859
@blckngm blckngm deleted the smt-trie-db branch March 14, 2023 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants