KAFKA-19499: Corrected the logger name in PersisterStateManagerHandler

[chirag-wadhwa5]

This PR corrects the name of logger in the inner class
PersisterStateManagerHandler. After this change it will be possible to
change the log level dynamically in the kafka brokers. This PR also
includes a small change in a log line in GroupCoordinatorService, making
it clearer.

Reviewers: Apoorv Mittal apoorvmittal10@gmail.com, Sushant Mahajan smahajan@confluent.io, Lan Ding isDing_L@163.com, TengYao Chi frankvicky@apache.org

[apoorvmittal10]

@chirag-wadhwa5 The build is failing with Persister tests, can you please fix.

[chirag-wadhwa5]

Thanks for the review @apoorvmittal10 . I have pushed the required changes. The new build report has only unrelated failures.

[DL1231]

Thanks for the patch, LGTM.
nit: "it will be possible t" -> "it will be possible to"

[frankvicky]

@chirag-wadhwa5: This patch LGTM.
Could you please merge the trunk?
The failed test should have been fixed on the trunk..

[smjn]

Thanks for the PR, LGTM

[apoorvmittal10]

Minor nit, probably do it with some other commit in future, if you think that's right. LGTM!

[apoorvmittal10]

nit: probaly can do later as well. Becuase now it's not always the canonical name in the variable.

Suggested change

	String canonicalName = getClass().getCanonicalName();
	if (canonicalName == null) {
	canonicalName = getClass().getName();
	}
	log = LoggerFactory.getLogger(canonicalName);
	String name = getClass().getCanonicalName();
	if (name == null) {
	name = getClass().getName();
	}
	log = LoggerFactory.getLogger(name);