addressing comments

Signed-off-by: Sarthak Aggarwal <[email protected]>

commands/client-kill.md

@@ -13,7 +13,7 @@ instead of killing just by address. The following filters are available:
 
 * `CLIENT KILL ADDR ip:port`. This is exactly the same as the old three-arguments behavior.
 * `CLIENT KILL LADDR ip:port`. Kill all clients connected to specified local (bind) address.
-* `CLIENT KILL ID [client-id ...]`. Allows to kill a client by its unique `ID` field. Client `ID`'s can be retrieved using the `CLIENT LIST` command. The filter supports one or more `client-id` arguments
+* `CLIENT KILL ID client-id [client-id ...]`. Allows to kill a client by its unique `ID` field. Client `ID`'s can be retrieved using the `CLIENT LIST` command. The filter supports one or more `client-id` arguments
 * `CLIENT KILL TYPE type`, where *type* is one of `normal`, `master`, `replica` and `pubsub`. This closes the connections of **all the clients** in the specified class. Note that clients blocked into the `MONITOR` command are considered to belong to the `normal` class.
 * `CLIENT KILL USER username`. Closes all the connections that are authenticated with the specified [ACL](../topics/acl.md) username, however it returns an error if the username does not map to an existing ACL user.
 * `CLIENT KILL SKIPME yes/no`. By default this option is set to `yes`, that is, the client calling the command will not get killed, however setting this option to `no` will have the effect of also killing the client calling the command.

commands/client-list.md

@@ -1,7 +1,7 @@
 The `CLIENT LIST` command returns information and statistics about the client
 connections server in a mostly human readable format.
 
-You can use one or more optional subcommands to filter the list:
+You can use one or more optional arguments to filter the list:
 
 - **`TYPE type`**: Filters the list by clients' type, where *type* is one of `normal`, `master`, `replica`, and `pubsub`.
   > Note: Clients blocked by the `MONITOR` command belong to the `normal` class.
@@ -12,22 +12,17 @@ You can use one or more optional subcommands to filter the list:
 
 - **`ADDR ip:port`**: Filters the list to include only clients connected from the specified address.
 
-- **`LADDR ip:port`**: Filters the list to include only clients bound to the specified local address.
+- **`LADDR ip:port`**: Filters the list to include only clients connected to the specified local address.
 
 - **`SKIPME yes|no`**: Filters whether the list should skip the client making the request.
     - `yes`: Skips the client making the request.
     - `no`: Includes the client making the request.
 
 - **`MAXAGE milliseconds`**: Filters the list to include only clients whose connection age (time since the client was created) is greater than or equal to the specified number of milliseconds.
-
-#### Combining Filters
+  > Note: This is actually a minimum age, not a maximum age. This filter was first added to CLIENT KILL, where the intention was to keep clients of a maximum age and kill the ones than the max age.
 
 Filters can be combined to perform more precise searches. The command will handle multiple filters via logical AND.
 
-```bash
-CLIENT LIST TYPE normal USER admin MAXAGE 5000 ID 1234 5678
-```
-
 Here is the meaning of the fields:
 
 * `id`: a unique 64-bit client ID