Support ASKING command

[jjz921024]

Close #2168

Support ASKING command，but didn't do anythins. Because the kvrocks seems will serving the imported slot despite the lack of asking flag

kvrocks/src/cluster/cluster.cc

Lines 857 to 862 in ae5784d

	if (myself_ && imported_slots_.count(slot)) {
	// After the slot is migrated, new requests of the migrated slot will be moved to
	// the destination server. Before the central controller change the topology, the destination
	// server should record the imported slots to accept new data of the imported slots.
	return Status::OK(); // I'm serving the imported slot
	}

[git-hulk]

Support ASKING command，but didn't do anythins. Because the kvrocks seems will serving the imported slot despite the lack of asking flag

Yes, exactly. But we can fix this behavior after supporting the ASKING command. By the way, guess we should clear the ASKING flag after executing the next query?

[jjz921024]

Support ASKING command，but didn't do anythins. Because the kvrocks seems will serving the imported slot despite the lack of asking flag

Yes, exactly. But we can fix this behavior after supporting the ASKING command. By the way, guess we should clear the ASKING flag after executing the next query?

Yes, Fixing it will make kvorcks more like with redis in behavior. Let me do it.

[jjz921024]

@git-hulk Hi, done it, please review

[jjz921024]

I find that the kvrocks doesn't seem to return ASK error when query a key in migrating slot ?

[git-hulk]

@jjz921024 Thank you! Generally looks good to me, you can fix the Go format to pass the CI.

[jjz921024]

@jjz921024 Thank you! Generally looks good to me, you can fix the Go format to pass the CI.

Yes, I had fix it

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
61.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud