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

Create a new Java based server to handle user inputs #18

Open
OhmGeek opened this issue Oct 19, 2019 · 1 comment
Open

Create a new Java based server to handle user inputs #18

OhmGeek opened this issue Oct 19, 2019 · 1 comment
Assignees
@OhmGeek
Labels
enhancement New feature or request

Comments

@OhmGeek
Copy link
Collaborator

OhmGeek commented Oct 19, 2019

The existing Python based server works in simple cases, but there are several issues:

  • Everything uses REST, which doesn't scale well for the emulator side (since UDP would be preferable for the emulator)
  • Python dictionary can't easily handle concurrency. Replacing this with a ConcurrentHashMap would be ideal.

Therefore, we need to:

  • create a new server project (leaving the old one within the repo for now).
  • This project will be extended to add a REST controller, controller service, and the UDP controller in future issues.
@OhmGeek OhmGeek added the enhancement New feature or request label Oct 19, 2019
@OhmGeek OhmGeek self-assigned this Oct 19, 2019
OhmGeek added a commit that referenced this issue Oct 19, 2019
@OhmGeek
#18: create a new Java maven based project
15d92a9
OhmGeek added a commit that referenced this issue Oct 19, 2019
@OhmGeek
#18: create maven project with dependencies
c9f38ea
OhmGeek added a commit that referenced this issue Oct 19, 2019
@OhmGeek
#18: create basic models that are necessary for this to work
fbd7214
OhmGeek added a commit that referenced this issue Oct 19, 2019
@OhmGeek
#18: add a basic command service
b9c06a6
OhmGeek added a commit that referenced this issue Oct 19, 2019
@OhmGeek
#18: Add state and tests for ControllerState
aeb9511
@OhmGeek
Copy link
Collaborator Author

OhmGeek commented Oct 19, 2019

PR #19

OhmGeek added a commit that referenced this issue Oct 20, 2019
@OhmGeek
Merge pull request #19 from georgeprice/feature/#18-new-java-server
2e105ce 
Feature/#18 new java server
OhmGeek added a commit that referenced this issue Dec 21, 2019
@OhmGeek
#18: add vertx to the project for REST
394b5b0
OhmGeek added a commit that referenced this issue Dec 21, 2019
@OhmGeek
#18: transfer java to kotlin, and rewite a basic kotlin vertx server
9021dec
OhmGeek added a commit that referenced this issue Dec 29, 2019
@OhmGeek
#18: Kotlin works using Vertx for a basic RESTful API which doesn't a…
1fb488a 
…ctually do much yet
OhmGeek added a commit that referenced this issue Dec 29, 2019
@OhmGeek
#18: tidy up Kotlin injection code, and use a factory to create contr…
32c3a22 
…oller state
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@OhmGeek OhmGeek

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@OhmGeek