foundry
foundry copied to clipboard
Remove obsolete command line options
Check if the following options are still necessary:
~work-queue-size~
~no-reseal-timer~
~reseal-max-period~
reseal-min-periodreseal-on-txsforce-sealing
@majecty I checked all of these commands some of them are still in the code. Do I need to remove any usage of them?
For force-sealing, it uses in some of the tests. Can you explain what is the purpose of force-sealing?
@MSNTCS AFAIK, force-sealing is used to create blocks in solo consensus even though there are no transactions. I think we can remove it if we remove solo consensus. I know that some code written in the mold branch does not consider the solo consensus. So we may remove it when the mold code is merged in the master branch.
@MSNTCS Please check them if they are used in a meaningful way. If understanding them is difficult, please let this issue.
@majecty some of them can be easily removed but some of them need to be carefully removed. For the second one, I also need some knowledge which will take time to fully understand it.
This PR will remove some of the obsolete commands while the last two commands need more time to go over all their usage.