GC stats no longer being collected about cstar_perf HOT 11 CLOSED

Looks like JVM_OPTS has the necessary c.s.m.j.authenticate setting by default, if I read this correctly:

from cstar_perf.

But, you say you need to override JVM_EXTRA_OPTS -- why is that necessary? It looks like it's empty by default. This conditional is the only place it seems to be set, and it's not executed by default.

If I read this correctly, this should be fixed by setting LOCAL_JMX=no by default. Does setting that all by itself in the job config make GC stat collection work the way you want, @tjake?

This might all be moot though, because I'd like to talk about the possibility of reverting 35340bd. I don't understand the logic behind it. The changes mentioned in the commit message only affect 3.6, so why change defaults for running all versions @mshuler?

from cstar_perf.

Because sometimes users want to set that. Since its applied last it overrides any pre-sets.

No setting LOCAL_JMX=no doesn't work by itself. I tried and it didn't work. I needed both

from cstar_perf.

To be clear JVM_EXTRA_OPTS is used in cassandra-env.sh

from cstar_perf.

Because sometimes users want to set that. Since its applied last it overrides any pre-sets.

I think I understand what you're saying now, gotcha. So cstar_perf just ignores users' JVM_EXTRA_OPTS, huh?

@tjake What are your thoughts on reverting the commit that caused this? We may have to wait until next week to hear from Michael and make a decision on that.

from cstar_perf.

On 06/30/2016 10:44 AM, Jim Witschey wrote:

This might all be moot though, because I'd like to talk about the
possibility of reverting 35340bd
35340bd.
I don't understand the logic behind it. The changes mentioned in the
commit message only affect 3.6, so why change defaults for running all
versions @mshuler https://github.com/mshuler?

It has been a little time since I tested this out, but the setting was
removed because:

a) it did nothing, since it was redundant to the default
b) it logged a jmx bind failure on every pre-3.6 c* version I attempted
c) a new c* commit to 3.6 started taking that logged jmx bind failure
and made it a fatal startup error

Feel free to revert, although >= 3.6 c* will fail to start, unless some
other better/more correct configuration mechanism is used such as Jake
is suggesting.

Michael

from cstar_perf.

I think we need to fix this by adding appending to JVM_EXTRA_OPTS

from cstar_perf.

Like literally Add:

LOCAL_JMX=no
JVM_EXTRA_OPTS="$JVM_EXTRA_OPTS -Dcom.sun.management.jmxremote.authenticate=false"

At the end of the passed ENV

from cstar_perf.

I can see why the LOCAL_JMX change is necessary, but I don't understand the JVM_EXTRA_OPTS part -- since JVM_EXTRA_OPTS just gets added to JVM_OPTS in cassandra-env, and c.s.m.j.authenticate is already set to false in fab_common, why is setting it in JVM_EXTRA_OPTS necessary?

from cstar_perf.

Because later on its set to true which overrides your setting.

from cstar_perf.

hup, yeah, there it is:

https://github.com/apache/cassandra/blob/trunk/conf/cassandra-env.sh#L255

Ok, I see now. Making a PR now for Eduard/Michael's review

from cstar_perf.