[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-11-17 Thread dkuppitz
Github user dkuppitz commented on the issue: https://github.com/apache/tinkerpop/pull/439 I can't test Systemd, but I've installed Gremlin Server as a service and that worked like a charm. Manual starts / restarts / stops /status requests worked as expected. Killing the process didn't

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-11-14 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 @PommeVerte do you think you might have some free time soon to review this and vote? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 LGTM --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 @spmallette I pushed the windows changes. can you review? --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does no

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 Tested all the command line options and all work nicely. Good documentation. It did occur to me that we don't have analogous features for windows in that the gremlin-server.sh does not

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 Rebased. Retested. VOTE +1 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature e

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 I believe @PommeVerte is on a month long holiday at this point. imo, i think we could move the ubuntu script off to a different ticket as you have a lot of work in place here. I would just reba

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-27 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 @PommeVerte do you mind if we move the ubuntu script to a separate pull request so this one can move forward? --- If your project is set up for it, you can reply to this email and have your r

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-04 Thread PommeVerte
Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/439 I'm currently on some ressource intensive tasks so it's hard for me to bounce back and forth between projects but I would like to give this a run on ubuntu 14.04. I suspect that sysV might work

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-04 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 I just tested with backgrounding the job and killing that. That also still works. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as w

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-04 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 oh - right - you had said that. sorry --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-04 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 That should still work. What won't work is an invocation of just "gremlin-server.sh" without parameters. I didn't find any in the code base nor in the docs (which is highly convenient). --

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-10-04 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 It occurred to me that it would seem there must be a place that gremlin-server.sh is used in doc generation - and here it is: https://github.com/apache/tinkerpop/blob/8ab682fef3e3da12

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-30 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 Cool. I was wondering how to link. Didn't think it was absolute. I hope it works. Tried testing with `./docker/build.sh -d` but it hangs at installing plugins `* hadoop-gremlin ...`. --- If

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-30 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 wow - @robertdale - this is a really nice PR. I added the "breaking" label to the JIRA ticket as this would prevent gremlin-server.sh from working as it did before. I think I'm ok with that, bu

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-30 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 - `gremlin-server.sh` _is_ the init script. This makes it more inline with other Apache servers like Tomcat, ActiveMQ, solr, etc. - Moved how to install "As A Service" from Upgrade to Greml

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-28 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 Maybe if the script detects that it's called via link with no parameters, then it's in initd mode and displays help. If the script is called directly with no parameters, then it just runs in fo

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-28 Thread robertdale
Github user robertdale commented on the issue: https://github.com/apache/tinkerpop/pull/439 Upstart looks simple enough. I don't know if the one @dkuppitz created, referenced in TINKERPOP-980 , is still viable. But I can't test. Any volunteers? I would have to say that it's no

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-28 Thread spmallette
Github user spmallette commented on the issue: https://github.com/apache/tinkerpop/pull/439 I like the suggestion @PommeVerte made to have some docs in the gremlin-server section somewhere. Then you could just reference them from the upgrade docs. I don't know what an upstart

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-28 Thread PommeVerte
Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/439 Just confirming that although ubuntu 14.04 has `systemd` it does not have `systemd-as-init` . So an upstart would be a good idea and should be relatively easy since this PR seems to be thorough

[GitHub] tinkerpop issue #439: TINKERPOP-980: added sysv and systemd init scripts

2016-09-28 Thread PommeVerte
Github user PommeVerte commented on the issue: https://github.com/apache/tinkerpop/pull/439 Should we add upstart to this PR? I'm not sure if the systemd covers older versions of ubuntu. Namely 14.04LTS ? --- If your project is set up for it, you can reply to this email and have you