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

Support setting separate control address for precommit and provecommit #4894

Closed
jennijuju opened this issue Nov 17, 2020 · 3 comments
Closed
Labels
area/ux Area: UX P2 P2: Should be resolved

Comments

@jennijuju
Copy link
Member

This is similar to what we have for wdpost right now so that the commit messages are less likely to be stuck in the mpool when the network condition changes.

@jennijuju jennijuju added area/ux Area: UX P2 P2: Should be resolved labels Nov 17, 2020
@askender
Copy link

askender commented Dec 2, 2020

If miners forget to Increase the minimum piece/size of deal config, they may get hundreds of small file deals, PublishStorageDeals msgs will block other types of msgs.

If something is wrong, miners may need to do mpool-clear and noncefix to send hundreds of meaningless msgs to get the nonce right. It would be better to Support setting separate control address for precommit and provecommit

@f8-ptrk
Copy link
Contributor

f8-ptrk commented Dec 12, 2020

this seems to have been merged into master

https://filecoinproject.slack.com/archives/CG6P10H9T/p1607008924422200

works fine as far as i can tell

@jennijuju
Copy link
Member Author

resolved by #5103

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/ux Area: UX P2 P2: Should be resolved
Projects
None yet
Development

No branches or pull requests

3 participants