LindaSummer commented on issue #2421:
URL: https://github.com/apache/kvrocks/issues/2421#issuecomment-2241168168

   > > Hi @Beihao-Zhou ,
   > > I have read your proposal and have one concern on `Secondary Keys & 
Values > Label Index`. The key in proposal is in 
`[label_key]|[label_value]|[key]` format. It may need a full iteration scan 
when purging a key. Making key as the first prefix part like 
`[key]|[label_key]|[label_value]` may reduce the complexity on cleanup.
   > > I'm very interested in time series topic and willing to make some 
contribution in this feature. 😄
   > > Best Regards, Edward
   > 
   > The secondary indexing I proposed is mainly for`MRANGE`, which will do 
some filtering based on label. So for this command, the secondary index is 
pretty similar to tag field index in KQIR (#2329). If a key is purged, then I 
think constructing the corresponding secondary index key and deleting it would 
be enough?
   > 
   > Let me know if I misunderstand your question. Also feel free to use any 
idea from the proposal! <3
   
   Hi @Beihao-Zhou ,
   
   Thanks for your patience and response!
   I have read the KQIR encoding design and get your point.
   I will research more about the Redis's implementation.
   Please let me know if we have any new idea on this topic. 😊
   
   Best Regards,
   Edward


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to