Add rdb_transmitted to replstateToString so that we can see it in INFO

[enjoy-binbin]

In dual channel replication, when the rdb channel client finish
the RDB transfer, it will enter REPLICA_STATE_RDB_TRANSMITTED
state. During this time, there will be a brief window that we are
not able to see the connection in the INFO REPLICATION.

In the worst case, we might not see the connection for the
DEFAULT_WAIT_BEFORE_RDB_CLIENT_FREE seconds. I guess there is no
harm to list this state, showing connected_slaves but not showing
the connection is bad when troubleshooting.

Note that this also affects the valkey-cli --rdb and --functions-rdb
options. Before the client is in the rdb_transmitted state and is
released, we will now see it in the info (see the example later).

Before, not showing the replica info

role:master
connected_slaves:1

After, for dual channel replication:

role:master
connected_slaves:1
slave0:ip=127.0.0.1,port=21112,state=rdb_transmitted,offset=0,lag=0,type=rdb-channel

After, for valkey-cli --rdb-only and --functions-rdb:

role:master
connected_slaves:1
slave0:ip=127.0.0.1,port=0,state=rdb_transmitted,offset=0,lag=0,type=replica

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.42%. Comparing base (047080a) to head (79ffcae).
⚠️ Report is 19 commits behind head on unstable.

Additional details and impacted files

@@             Coverage Diff              @@
##           unstable    #2833      +/-   ##
============================================
+ Coverage     72.39%   72.42%   +0.02%     
============================================
  Files           128      128              
  Lines         70301    70328      +27     
============================================
+ Hits          50895    50934      +39     
+ Misses        19406    19394      -12     

Files with missing lines	Coverage Δ
src/server.c	88.43% <100.00%> (-0.03%)	⬇️

... and 15 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[JimB123]

Looks good. Missing state prevents view in INFO, but adding the state won't cause any excessive behavior.

[zuiderkwast]

What's the name of the INFO field?

Is it the field slave%d?

"slave%d:ip=%s,port=%d,state=%s,"
"offset=%lld,lag=%ld,type=%s\r\n"

[enjoy-binbin]

What's the name of the INFO field?
Is it the field slave%d?

yes, i forgot to add an example to the top comment before. I now have updated it and mentioned all the examples of the impact.