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

Re-implement threading logic to use coroutines #47

Merged
merged 63 commits into from
Jul 27, 2022
Merged

Re-implement threading logic to use coroutines #47

merged 63 commits into from
Jul 27, 2022

Conversation

hopskipnfall
Copy link
Owner

Also use ktor as a lightweight framework.

@hopskipnfall hopskipnfall mentioned this pull request Jun 26, 2022
Merged
@hopskipnfall
Add a server startup test.
2edb306
@hopskipnfall
Maybe helpful.
4a42bfe
@hopskipnfall
Fake kaillera client code for intensive testing (#48)
def6ec2 
* Add some rough fake client code for stress testing

* Working fake client and sample code for joining a common game.

* Comment out some debugging log lines.

* It works if i start, but throws an exception when someone joins a game that the bot started.

* Working

* Remove some log lines
@hopskipnfall
Fix some nullness stuff
92a6503
@hopskipnfall
Remove manual threading logic and replace with coroutines and ktor
334f8a7
@hopskipnfall
Remove manual threading logic and replace with coroutines and ktor
65c52a3
@hopskipnfall
Remove unnecessary run block.
bcb28de
@hopskipnfall
Add a server startup test.
e90f032
@hopskipnfall
Maybe helpful.
7155420
@hopskipnfall
ugh git sucks
1782f37
@hopskipnfall
Fix some types and use lateinit
559e377
@hopskipnfall
format
58f6c07
@hopskipnfall
Some nullability stuff, and add a flogger extension method
3952b9e
@hopskipnfall
Add a test to make sure users can create games
0a09f56
@hopskipnfall
Refactor waiting logic
83cd067
@hopskipnfall
Nullness!
c8b135d
@hopskipnfall
Logging optimizations
484bdff
@hopskipnfall
wooooOOOO!
92dbcf8
@hopskipnfall
zzzzz
ccd3410
@hopskipnfall
More cleanups
f6205e5
@hopskipnfall
Probably fix synchronization issues for ConnectController.
7ff83d7
@hopskipnfall
Fix AccessManager to use coroutines
a941021
@hopskipnfall
please help
cffa519
@hopskipnfall
0.6.2
528fc1b
@hopskipnfall
Probably fix the resend deadlock issue.
0b3ac79
@hopskipnfall
Fix some nullness stuff
7e4324a
@hopskipnfall hopskipnfall reopened this Jul 14, 2022
@hopskipnfall
Copy link
Owner Author

OK this is in a working state, but there's still more coroutine-related work to be done:

  • Use a Channel which is more coroutine-friendly instead of a BlockingQueue for game data
  • Replace GlobalScope usages with some known context scoped to the user
  • Make sure we're using the right Dispatcher everywhere

@hopskipnfall
Copy link
Owner Author

Actually a few other places like the admin server also require updating I think. but this is good enough for one PR.

@hopskipnfall hopskipnfall merged commit 988317c into master Jul 27, 2022
@hopskipnfall hopskipnfall deleted the clienthandlerstuff branch July 27, 2022 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@hopskipnfall