Skip to content

Make dynamic gas price suggestion compatible with parallel execution#2829

Merged
KamiD merged 60 commits intodevfrom
fix-conflict165
Dec 8, 2022
Merged

Make dynamic gas price suggestion compatible with parallel execution#2829
KamiD merged 60 commits intodevfrom
fix-conflict165

Conversation

@KamiD
Copy link
Member

@KamiD KamiD commented Dec 1, 2022

  • Targeted PR against correct branch (see CONTRIBUTING.md)

  • Linked to github-issue with discussion and accepted design OR link to spec that describes this work.

  • Wrote tests

  • Updated relevant documentation (docs/)

  • Added a relevant changelog entry to the Unreleased section in CHANGELOG.md

  • Reviewed Files changed in the github PR explorer


For Admin Use:

  • Added appropriate labels to PR (ex. wip, ready-for-review, docs)
  • Reviewers Assigned
  • Squashed all commits, uses message "Merge PR #XYZ: [title]" (coding standards)

lyh169 and others added 30 commits November 3, 2022 14:37
* delete feesplit innertx

* Update keeper_innertx.go
2. fix the --deliver-txs-mode=2 can not query status
3. update the version
when repair-state we judge the fast-storage flag by data

add feesplit module

reserve --iavl-enable-fast-storage flag for repair-state

merge repair-state we judge... and deal with confict

rm NeedLongTimeToUpgrade use the IsFastStorgeStrategy universal

Co-authored-by: xiangjianmeng <805442788@qq.com>
* roll back blocktime

* del flag
* when start with enable-repair-state and the db is fast-storage, we repair with the iavl-enable-fast-storage

* adjust the fast-storage node size
Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
* set chain-id default to exchain-66

* set EnablePrerunTx default to true

* set pruning default to everything

* enable ac default

* disableABCIQueryMutex default

* enable fast-query by default

* enable BloomFilter by default

* update mempool Size default value

* update val mode config

* add tx_indexer.indexer cmd flags

* add enable-concurrency flag

* set chainid to exchain65 if make testnet
* gp bug fix

* modify code

* add flag DynamicGpAdaptUncongest

* modify ut

* adapt congest

* bug fix

* code optimize

Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
* optimize start params

* optimize start param

* optimize start param

* optimize start param
* gp bug fix

* modify code

* add flag DynamicGpAdaptUncongest

* modify ut

* adapt congest

* bug fix

* code optimize

* add flag: gp maxGas maxTx

* Update config.go

* Update flags.go

Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
* gp bug fix

* modify code

* add flag DynamicGpAdaptUncongest

* modify ut

* adapt congest

* bug fix

* code optimize

* add flag: gp maxGas maxTx

* bug fix

* modify flag

* typo fix
* fast storage is default true and set by the data

* add fast-storage tips when start the node

* hidden flag discard-fast-storage

* print iavl.IsFastStorage when start node

* Update Makefile

* restore and hide the old flag

Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
* 1. add commitGapHeight for dynamic update

* 1. add comment

* 1. add UpdateCommitGapHeight at before commitStores

* 1. add UpdateCommitGapHeight at before commitStores

* 1. modify the spell

Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
* support dynamic set fast node cache size

* use atomic operate the size

* do not use atomic

* use one const and reduce dup flag
* modify dynamic gp flag

* fix bug

* rewrite ut

* magic number fix

* replace flag in sh

* modify code

* fix typo

* mark hidden enable-dynamic-gp

* improve code

* improve code
* add debug test

* preblock test

* restore producer

* adjust channel buffer

Co-authored-by: xiangjianmeng <805442788@qq.com>
* Update app_parallel.go

first. commit

Update baseapp_runtx.go

Update abci.go

1

Update app_parallel.go

Update app_parallel.go

Update baseapp_mode_deliver.go

1

Update evm_hooks.go

Update baseapp_mode_deliver.go

Update base.go

Update evm_hooks.go

1

a

Update repair_state.go

Revert "Update repair_state.go"

This reverts commit 3051e30.

* fix test

* typ

* add check

* add check

Co-authored-by: xiangjianmeng <805442788@qq.com>
* applybolck workload statistic

* complete workload statistic

* fix bug  & change format

* remove BlockExecutor field

* fix atomic on lower go version

* remove assert

* make it more accurately if process started less than 1 hour

* make code easier to understand

* fix code style

Co-authored-by: Zhong Qiu <36867992+zhongqiuwood@users.noreply.github.com>
Co-authored-by: yangzhe <zhe.yang@okg.com>
chengzhinei and others added 13 commits November 27, 2022 22:09
* fix config of timeout_commit

* fix config fmt

* add db_backend

* add pendingPool=false

* del setEnv rocksdb

* unify timeout_commit define in one place

* testnet.sh

* add log

* add log

Co-authored-by: xiangjianmeng <805442788@qq.com>
* support multi-cache when repair state

* add pprof

* fix net pprof
* add enable config of hgu

* simulate tx in new goroutine

* add simulation debug info

* start more goroutine for simulation

* fix bug

* udpate

* fix bug of sig cache

* disable pendingPool

* enable hgu default

* add log info and query api of simulation gas

* add enable config of pgu and adjustment

* use atomic to prevent data race

* delete temp code

* enable async simulation only when pgb is greater than -1

* check error
* fix websocket leak

* optimize code
* remove the lock when get fast node

* replace mutex with rbmutex
* version 1

* bug fix

* rename mode and set MinimalGpMode for default

* rewrite ut

* delete the invalid annotations

* improve code

* enable dynamic config

* improve code

* improve code

Co-authored-by: KamiD <44460798+KamiD@users.noreply.github.com>
@KamiD KamiD changed the title Make gas price suggestion compatible with parallel execution Make dynamic gas price suggestion compatible with parallel execution Dec 1, 2022
xiangjianmeng
xiangjianmeng previously approved these changes Dec 1, 2022
@codecov-commenter
Copy link

codecov-commenter commented Dec 1, 2022

Codecov Report

Merging #2829 (b8b8af1) into dev (754b423) will increase coverage by 0.03%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##              dev    #2829      +/-   ##
==========================================
+ Coverage   36.30%   36.33%   +0.03%     
==========================================
  Files         415      415              
  Lines       43814    43808       -6     
==========================================
+ Hits        15907    15918      +11     
+ Misses      26752    26736      -16     
+ Partials     1155     1154       -1     
Impacted Files Coverage Δ
app/app_abci.go 58.33% <ø> (+3.01%) ⬆️
app/node_mode.go 8.64% <0.00%> (ø)
app/rpc/namespaces/eth/api.go 0.00% <0.00%> (ø)
app/types/gasprice.go 0.00% <ø> (ø)
app/app.go 81.52% <100.00%> (ø)
app/app_parallel.go 48.10% <100.00%> (+18.37%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@KamiD KamiD marked this pull request as draft December 1, 2022 16:00
@LeoGuo621 LeoGuo621 marked this pull request as ready for review December 1, 2022 18:22
@KamiD KamiD merged commit ebe8d7b into dev Dec 8, 2022
@KamiD KamiD deleted the fix-conflict165 branch December 8, 2022 02:31
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.