8000 Bugfix/fix test arangod shutdown command by dothebart · Pull Request #3260 · arangodb/arangodb · GitHub
[go: up one dir, main page]
Skip to content

Bugfix/fix test arangod shutdown command #3260

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

Merged
merged 4 commits into from
Sep 26, 2017
Merged

Bugfix/fix test arangod shutdown command #3260

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
551cea1
If our arango-object claims its got a connection to the arangodb serv…
dothebart Sep 14, 2017
fa7bf22
don't use kill to shut down the SUT, we now can use the regular shutd…
dothebart Sep 14, 2017
efb4198
lint
dothebart Sep 15, 2017
6ee35bc
call function instead of testing for its existence
dothebart Sep 15, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
call function instead of testing for its existence
  • Loading branch information
@dothebart
dothebart committed Sep 15, 2017
commit 6ee35bc5f82dff7fa5baac6aea55e44858b76f62
2 changes: 1 addition & 1 deletion js/client/modules/@arangodb/process-utils.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -758,7 +758,7 @@ function shutdownArangod (arangod, options, forceTerminate) {
} else if (options.useKillExternal) {
killExternal(arangod.pid);
} else {
if (arango.isConnected) {
if (arango.isConnected()) {
try {
arango.DELETE('/_admin/shutdown');
if (options.extremeVerbosity) {
Expand Down
0