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

Any potential runtime issue to run multiple datart-server on same box? #1126

Closed
clementchong opened this issue Mar 31, 2022 · 6 comments · Fixed by #1144
Closed

Any potential runtime issue to run multiple datart-server on same box? #1126

clementchong opened this issue Mar 31, 2022 · 6 comments · Fixed by #1144
Labels
good first issue Good for newcomers

Comments

@clementchong
Copy link
Contributor

Any issue if we have multiple datart running on same server with different ports? Databases are on different schemas but same mysql instance and port. datart.conf would have a different database value, e.g

datasource.database=datart_1

datasource.database=datart_2

Do we have code, reference, token, etc conflict issue?

@scottsut
Copy link
Contributor

scottsut commented Apr 1, 2022

It is possible to deploy multiple datart instances in one server, but now there are some issues with the startup script, so that only one instance can actually be started, you can do it by modifying the startup script logic.

@scottsut scottsut added the good first issue Good for newcomers label Apr 1, 2022
@clementchong
Copy link
Contributor Author

Ok, would try out and share script if successful

@clementchong
Copy link
Contributor Author

I think you are referring to the linux shell script. The windows one runs fine and able to support multiple datart instances on same box.

@scottsut
Copy link
Contributor

scottsut commented Apr 1, 2022

I think you are referring to the linux shell script. The windows one runs fine and able to support multiple datart instances on same box.

Thank you for the feedback on windows. datart-server.sh caused the problem.

@clementchong
Copy link
Contributor Author

PR submitted

@clementchong
Copy link
Contributor Author

Yay! PR approved and merged.

@scottsut scottsut linked a pull request Apr 22, 2022 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants