Python: adds JSON.RESP command

[shohamazon]

Issue link

This Pull Request is linked to issue (URL): #645

Checklist

Before submitting the PR make sure the following are checked:

• This Pull Request is related to one issue.
• Commit message has a detailed description of what changed and why.
• Tests are added or updated.
• CHANGELOG.md and documentation files are updated.
• Destination branch is correct - main or release
• Commits will be squashed upon merging.

[Yury-Fridlyand]

Some of my comments caused by improper understanding of how command works.

> JSON.RESP k1 .address
1) {
2) 1) "street"
   2) "21 2nd Street"
3) 1) "city"
   2) "New York"
4) 1) "state"
   2) "NY"
5) 1) "zipcode"
   2) "10021-3100"

That what's GLIDE core should return:

1# "street" => "21 2nd Street"
2# "city" => "New York"
3# "state" => "NY"
4# "zipcode" => "10021-3100"

and a GLIDE client should return a map/dict then.
Otherwise we violate data integrity on client-server-client round trip.

Example (expected):

object = {"firstName":"John","lastName":"Smith","age":27,"address":{"street":"21 2nd Street","city":"New York","state":"NY","zipcode":"10021-3100"}}
await json.set(glide_client, key, ".", OuterJson.dumps(json_value))

response = await json.resp(glide_client, key, ".address")

assert response == object["address"]

[shohamazon]

Some of my comments caused by improper understanding of how command works.

> JSON.RESP k1 .address
1) {
2) 1) "street"
   2) "21 2nd Street"
3) 1) "city"
   2) "New York"
4) 1) "state"
   2) "NY"
5) 1) "zipcode"
   2) "10021-3100"

That what's GLIDE core should return:

1# "street" => "21 2nd Street"
2# "city" => "New York"
3# "state" => "NY"
4# "zipcode" => "10021-3100"

and a GLIDE client should return a map/dict then. Otherwise we violate data integrity on client-server-client round trip.

Example (expected):

object = {"firstName":"John","lastName":"Smith","age":27,"address":{"street":"21 2nd Street","city":"New York","state":"NY","zipcode":"10021-3100"}}
await json.set(glide_client, key, ".", OuterJson.dumps(json_value))

response = await json.resp(glide_client, key, ".address")

assert response == object["address"]

I dont understand why? we have json.get for this

[Yury-Fridlyand]

As discussed with the team, no value conversion needed.

[Yury-Fridlyand]

please return \ns from docs.

[shohamazon]

As discussed with the team, no value conversion needed.

ohh missed that

[avifenesh]

There's no external docs to refer to?

[shohamazon]

nope, valkey doesnt have docs for the modules commands
we do have the aws docs tho https://docs.aws.amazon.com/AmazonElastiCache/latest/dg/json-resp.html

[avifenesh]

Lets add, won't hurt.

[shohamazon]

will add it later with the rest of the commands, as no one has any external doc currently

[avifenesh]

Can we randomize it a bit?

[avifenesh]

One comment. Looks good!!