Yes that is correct, the ports are going to collide. I am suggesting that we should make the remote debugging enabled by default but have the other property for the shell command which does not collide with the generic properties
On Sun, Jun 15, 2014 at 7:58 PM, Josh Elser <[email protected]> wrote: > I think Vicky is just noting that each process would have to be defined > individually and not use ACCUMULO_GENERAL_OPTS as the debugger ports would > collide. > > I think making an ACCUMULO_SHELL_OPTS is fine. > On Jun 15, 2014 8:38 AM, "William Slacum" <[email protected]> > wrote: > > > Putting the flag in the process/module OPTs is fine. It's what I normally > > do when I want to debug. Are you suggesting we have remote debugging > > enabled by default? > > > > > > On Sun, Jun 15, 2014 at 9:11 AM, Vicky Kak <[email protected]> wrote: > > > > > While trying to get the remote debugger running with accumulo I figured > > > that for the accumulo shell command we need to introduce the following > > > changes > > > > > > 1) test -z "$ACCUMULO_SHELL_OPTS" && export > > ACCUMULO_SHELL_OPTS="-Xmx128m > > > -Xms64m -Xrunjdwp:server=y,transport=dt_socket,address=4002,suspend=n" > > > > > > in accumulo-env.sh > > > > > > 2) > > > include the additional case in accumulo.sh > > > > > > shell) export ACCUMULO_OPTS="${ACCUMULO_GENERAL_OPTS} > > > ${ACCUMULO_SHELL_OPTS}" ;; > > > > > > We can't define the debugger port in the $ACCUMULO_OTHER_OPTS in the > > > accumulo-env.sh as that would be bind when start-all.sh is called. > > > > > > Before I raise a JIRA for this a provide a patch I would like to hear > the > > > option from others how they enable the remote debugging for the shell > > > command. > > > > > > Thanks, > > > Vicky > > > > > >
