dice icon indicating copy to clipboard operation
dice copied to clipboard
verified publisher icon DiceDB

Added support for JSON.NUMMULTBY

Open c-harish opened this issue 1 year ago • 4 comments

Implementation for issue 497

Added support for JSON.NUMMULTBY command. It multiplies existing fields in JSON that matches specified JSON path at the specified key by the multiplier value. Returns null for non numerical fields. Output is stringified array of updated values.

redis documentation : JSON.NUMMULTBY

Changes :

  • evalJSONNUMMULTBY implementation
  • testevalJSONNUMMULTY unit test
  • TestJsonNummultby integration test
image

c-harish avatar Sep 15 '24 18:09 c-harish

@c-harish please rebase the PR once, to resolve conflicts.

lucifercr07 avatar Sep 16 '24 13:09 lucifercr07

@JyotinderSingh
I have rebased. Please review and let me know your suggestions.

c-harish avatar Sep 17 '24 16:09 c-harish

@c-harish please resolve conflicts.

lucifercr07 avatar Sep 18 '24 20:09 lucifercr07

@c-harish please resolve conflicts.

rebased, @lucifercr07. please review.

c-harish avatar Sep 19 '24 04:09 c-harish