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.

Sign up for GitHub

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

Jump to bottom

[fastx client] Improve the quorum_map_then_reduce_with_timeout function #377

Closed
gdanezis opened this issue Feb 6, 2022 · 2 comments
Closed

[fastx client] Improve the quorum_map_then_reduce_with_timeout function #377

gdanezis opened this issue Feb 6, 2022 · 2 comments
Labels
Type: Enhancement New feature or request

Comments

@gdanezis
Copy link
Collaborator

gdanezis commented Feb 6, 2022

There is a specific suggestion here:
#336 (comment)
@gdanezis gdanezis added the Type: Enhancement New feature or request label Feb 6, 2022
@gdanezis gdanezis mentioned this issue Feb 6, 2022
Merged
@huitseeker
Copy link
Contributor

The comment was meant as "I tried and it failed", namely approaching the reducer as an FnMut leads nowhere: https://gist.github.com/b2c59c4f3cf014debbc81616844c0755

The lingering issue is to convert the reducer back to an Fn argument, which will be solved by #378

@gdanezis
Copy link
Collaborator Author

gdanezis commented Feb 7, 2022

Many thanks, closing!

@gdanezis gdanezis closed this as completed Feb 7, 2022
mwtian pushed a commit that referenced this issue Sep 12, 2022
@asonnino
Epoch numbers (#377)
edb455a 
Add epoch numbers to messages
mwtian pushed a commit to mwtian/sui that referenced this issue Sep 29, 2022
@asonnino
Epoch numbers (MystenLabs#377)
54872d0 
Add epoch numbers to messages
@Daywalker99 Daywalker99 mentioned this issue Nov 10, 2022
Open
@snyk-bot snyk-bot mentioned this issue Nov 20, 2022
Open
@snyk-bot snyk-bot mentioned this issue Apr 16, 2023
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: Enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@huitseeker @gdanezis