-
Notifications
You must be signed in to change notification settings - Fork 254
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
[SUBMARINE-192] Submarine workbench server daemon shell script #20
Conversation
bin/common.sh
Outdated
|
||
WORKBENCH_CLASSPATH+=":${SUBMARINE_CONF_DIR}" | ||
|
||
function addEachJarInDir(){ |
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.
Refer the Google Style (Function Names), it's' not acceptable.
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.
Done.
echo "${WORKBENCH_NAME} is restarting" >> "${WORKBENCH_LOGFILE}" | ||
stop | ||
start | ||
;; |
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.
What's the difference between reload
and start
command?
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.
No different, So i delete reload
commond.
bin/workbench-daemon.sh
Outdated
} | ||
|
||
function foundWorkbenchServerPid() { | ||
process=WorkbenchServer; |
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.
process=WorkbenchServer; | |
process='WorkbenchServer'; |
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.
Done.
bin/workbench-daemon.sh
Outdated
|
||
function foundWorkbenchServerPid() { | ||
process=WorkbenchServer; | ||
RUNNING_PIDS=$(ps x | grep $process | grep -v grep | awk '{print $1}'); |
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.
RUNNING_PIDS=$(ps x | grep $process | grep -v grep | awk '{print $1}'); | |
RUNNING_PIDS=$(ps x | grep ${process} | grep -v grep | awk '{print $1}'); |
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.
Done.
@jiwq , thank you advise. |
What is this PR for?
Increase the Submarine workbench server daemon script,
Support to start, stop, restart the submarine workbench server through
submarine-daemon.sh
What type of PR is it?
[Feature]
What is the Jira issue?
How should this be tested?
Screenshots (if appropriate)
Questions: