json.arrpop forces index=-1 with root path

[rigazilla]

Current Behavior

redis.jsonArrpop("k",new JsonPath("$"),i);

returns last element instead of element at pos i

Environment

• Lettuce version(s): [e.g. 6.5.5.RELEASE]

Possible Solution

Should be an easy fix, see current code here:

lettuce/src/main/java/io/lettuce/core/RedisJsonCommandBuilder.java

Lines 109 to 118 in cb02888

	if (jsonPath != null && !jsonPath.isRootPath()) {
	// OPTIONAL as per API
	args.add(jsonPath.toString());
	if (index != -1) {
	// OPTIONAL as per API
	args.add(index);
	}
	}

Additional context

[tishun]

🤦 embarrassing

Will address this in the next patch release

[thachlp]

I created the fix for this issue, please help review @tishun @rigazilla 🙇