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

fix submit thrashing #22473

Merged
merged 3 commits into from Feb 10, 2020
Merged

fix submit thrashing #22473

merged 3 commits into from Feb 10, 2020

Conversation

@chrisnojima
Copy link
Contributor

chrisnojima commented Feb 10, 2020

  • fix issue with submit() returning a new callback each time
  • add useMemo(one) to util/memo
chrisnojima added 2 commits Feb 10, 2020
WIP
@chrisnojima chrisnojima changed the title WIP: fix submit thrashing fix submit thrashing Feb 10, 2020
@chrisnojima chrisnojima marked this pull request as ready for review Feb 10, 2020
@mmaxim
mmaxim approved these changes Feb 10, 2020
WIP
@chrisnojima chrisnojima merged commit 5d7274d into master Feb 10, 2020
2 checks passed
2 checks passed
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/jenkins/pr-head This commit looks good
Details
@chrisnojima chrisnojima deleted the nojima/HOTPOT-use-rpc-fix branch Feb 10, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.