- Notifications
You must be signed in to change notification settings - Fork5
chore: create & delete sync sessions over gRPC#119
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Conversation
ethanndickson commentedMar 24, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
c11bc07 to1cac28dCompare84e4daa to4581c1eCompare1cac28d to015cd0cCompare4581c1e to7ca5e28Compare015cd0c to6a40c66Compare7ca5e28 to2b848e9Compare6a40c66 to257499fCompare2b848e9 to94f7e77Compare257499f to12fa59fCompare94f7e77 toe6b0bc3Compare12fa59f toaf6ba71Comparee6b0bc3 to080528aCompareaf6ba71 toae06ba2Comparec215f63 toa27014aCompare| return | ||
| } | ||
| group=MultiThreadedEventLoopGroup(numberOfThreads:1) | ||
| group=MultiThreadedEventLoopGroup(numberOfThreads:2) |
ethanndicksonMar 24, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
We always have two outgoing gRPC requests at once, the prompter, and the actual request we're trying to make. This creates a second OS thread for gRPC to use.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
tbh 2 os threads might be overkill
ae06ba2 to68dae7dComparea27014a to126e84fCompare68dae7d toc7018ecCompare126e84f toba8bd98Comparec7018ec to820baceComparebc0d1e3 to660e1dfCompare032237c toeb29807Compare820bace to93667f7Compareeb29807 to58c4ff5Compareethanndickson commentedMar 28, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
58c4ff5 to58f9775Compare6463de0 intomainUh oh!
There was an error while loading.Please reload this page.

Uh oh!
There was an error while loading.Please reload this page.
Closes#63.