Fix the gson dependency in the kafka-reporter-plugin. (#10848)

[gzlicanyi]

• If this pull request closes/resolves/fixes an existing issue, replace the issue number. Closes [Bug] kafka-reporter-plugin-8.15.0 throw exception "java.lang.NoClassDefFoundError: com/google/gson/Gson" skywalking#10848.
• Update the CHANGES log.

[lujiajing1126]

I don't think we need to explicitly add this dependency since we've already included apm-agent-core

[gzlicanyi]

This is old code. Similar to Gson, the kafka-reporter-plugin does not use any dependencies from apm-agent-core.

[lujiajing1126]

This is old code. Similar to Gson, the kafka-reporter-plugin does not use any dependencies from apm-agent-core.

IIRC, the shaded Gson library is already located in the agent-core jar. So just shading this library at compile-time is sufficient.

Just like what we've done for slf4j.

[wu-sheng]

This is old code.

This is something you added, this is not old code.

[gzlicanyi]

This is old code. Similar to Gson, the kafka-reporter-plugin does not use any dependencies from apm-agent-core.

IIRC, the shaded Gson library is already located in the agent-core jar. So just shading this library at compile-time is sufficient.

Just like what we've done for slf4j.

You are right, I will give it a try.

[wu-sheng]

We don't need this. The property name is even longer than the original package name, there is no point.

[wu-sheng]

And, I don't think we are expecting shading all com.google. You are just going to shade GSON relative things. Please be clear about that.

[gzlicanyi]

And, I don't think we are expecting shading all com.google. You are just going to shade GSON relative things. Please be clear about that.

Let me optimize this.

[wu-sheng]

This should be good. @lujiajing1126 It is better you could check locally.

[lujiajing1126]

LGTM. I've compiled locally and everything is fine.