kushal0511-not
kushal0511-not
@JyotinderSingh if nobody is working on this , can i take this one? cc @AshwinKul28
[@JyotinderSingh](https://github.com/JyotinderSingh) I am thinking of adding cleanup method in stackref.go ``` // Cleanup removes expired or deleted keys from the stack. func (s *StackRef) Cleanup() { var validElements []int64 for...
@NOS-AE Thanks for suggestions. > returning nil when STACKREFPOP a deleted key This will change current behavior of Pop(), that is popping until existing non-expired key. @JyotinderSingh which approach should...
@JyotinderSingh In worst case for 1st approach could be like many (suppose 10000 ) expired key and 2-3 non-expired key. then STACKREFLEN return a number which will far from real...
Going with the cleanup method.
> Let me know what you both think, if you agree with this approach we can discuss about the implementation. @soumya-codes Sure we can discuss. I have doubt about how...
@soumya-codes and I discussed this issue and we've reached the following conclusions: 1. There is no reference in redis of QREF or STCKREF from which we can get to know...
> > STACKREF is a novel data structure in dice. I am okay with removing the STACKREFLEF command. Alternatively we could maintain the command with the performance penalty of clearing...
@arpitbbhayani can i take this up?
We have some inconsistency in command due to how we treat **JSONPath** in JSON.SET command. here are few examples. 1 ) dice :  Redis 7.2.5 :...