[Issue 10669] Add Pulsar_GC_LOG when start broker with bin/pulsar

[Jason918]

Fixes #10669

Motivation

GC log is useful for production environment, I think it should take affect by default.

[jiazhai]

/pulsarbot run-failure-checks

[jiazhai]

@merlimat @codelipenghui What do you think of this change?

[hangc0276]

/pulsarbot run-failure-checks

[hangc0276]

@Jason918 For bookie startup, it will be better to add gc log too. In my production env, i have add the gc log for broker, zookeeper and bookie.

[jiazhai]

/pulsarbot run-failure-checks

[jiazhai]

@Jason918 Would you please help update your branch with the latest master? seems the always CI get failed.

[Jason918]

@Jason918 For bookie startup, it will be better to add gc log too. In my production env, i have add the gc log for broker, zookeeper and bookie.

OK, Added gc log for bookie in the updates.

[Jason918]

@Jason918 Would you please help update your branch with the latest master? seems the always CI get failed.

OK, merged with master.
BTW, Is it my commits causing the CI failed? These updates seem trivial.

[codelipenghui]

I agree with the motivation of the PR, it's useful for the production environment. Since we are trying to change the default value, it will make difference in the new Pulsar release, I think it's better to discuss this at the dev email thread first.