vert-x3 / vertx-stack

Vert.x stack
Apache License 2.0
120 stars 48 forks source link

Update scripts to enable vertx-sync #19

Closed purplefox closed 9 years ago

purplefox commented 9 years ago

We should update the vertx scripts so they (optionally) allow vertx-sync to be easily enabled.

This means adding the javaagent as described here https://github.com/vert-x3/vertx-sync/blob/master/README.adoc

Also the debugging options.

cescoffier commented 9 years ago

The user can configure $JAVA_OPTS or $VERTX_OPTS for this.

purplefox commented 9 years ago

True, but maybe we should make it simpler for them? Adding the javaagent path can be tricky and prone to error as it will have to reference the jar with the correct version in the lib directory.

We could change the script so it checked for a simple environment variable(e.g. ENABLE_SYNC) and if set then the script could add the -javaagent arg with the correct path.