fix: forbid DFLYCLUSTER commads set for emulated cluster mode

[BorysTheDev]

No description provided.

[adiholden]

as discussed find why test_replicate_disconnect_cluster is failing and fix it before you merge the PR

[chakaz]

Suggested change

	if (cntx->conn()) {
	VLOG(2) << "Got DFLYCLUSTER command (" << cntx->conn()->GetClientId() << "): " << args;
	} else {
	VLOG(2) << "Got DFLYCLUSTER command (NO_CLIENT_ID): " << args;
	}
	auto print_conn = [](auto* conn) { return conn ? conn->GetClientId() : "NO_CLIENT_ID" };
	VLOG(2) << "Got DFLYCLUSTER command (" << print_conn(cntx->conn()) << "): " << args;

[BorysTheDev]

it's different types. int and char[]

[chakaz]

You could return a string.. no need to duplicate code
(a string < 16 bytes will not do dynamic allocations)

[BorysTheDev]

Will change it next time when do some changes in this file

[adiholden]

In which flow do you get null connection?

[adiholden]

oh this is the journal emulated mode

[adiholden]

nit:
client_info = cntx->conn()? ntx->conn()->GetClientId() : "stub";
VLOG(2) << "Got DFLYCLUSTER command (" << client_info << args;

[BorysTheDev]

in journal_emulated