-
Notifications
You must be signed in to change notification settings - Fork 28
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
TODO: newPendingTransactions
and syncing
are not implemented in the fullnode yet.
#122
Comments
Hi, @waelsy123 Thanks for the interest in contributing to Conflux is EVM-compatible by providing EVM space. For now, So please file your issue to our conflux-rust repo for more details. |
Hi @wanliqun I just created another issue on conflux-rust: #issue 2672 . let me know please what do you think? and I would be available to implement the feature however I just need some technical guidance / onboarding to achieve the mentioned requirements. :) |
@wanliqun is the pending transactions pool being synced among nodes? Do other nodes remain unaware of a transaction until it has been committed? |
@waelsy123 Yes, once a transaction submitted to a node, the transaction would be replicated to other nodes within the same cluster group. |
I would like to implement this missing feature:
// eSpace PubSub notification
// TODO:
// 2.
newPendingTransactions
andsyncing
are not implemented in the fullnode yet.could you please provide more details on what needs to be done?
I'm not sure if in general other fullnode send these information so I can retrieve it and provide it by PubSub
The text was updated successfully, but these errors were encountered: