GitHub user git-hulk added a comment to the discussion: TimeSeries Proposal

Thanks for your great proposal.

For the encoding part, I think it would be better to encode the key type 
instead of introducing a new column family, which will allow users to rollback 
if have any issues in newer versions.

```
               
+------------------+----------------+------------------+------------------+
key|version => |  label_key1_size |  label_key1    |label_value1_size |  
label_value1    |
               |     (4byte)      |   (Xbyte)      |   (4byte)        |     
(Ybyte)      | 
               
+------------------+----------------+------------------+------------------+
               
+------------------+----------------+------------------+------------------+
               |  label_key2_size |  label_key2    |label_value2_size |  
label_value2    |...
               |     (4byte)      |   (Zbyte)      |   (4byte)        |     
(Sbyte)      |...
               
+------------------+----------------+------------------+------------------+
```

For the secondary index, do you mean that it will put all of label indexes in 
one key-value?


GitHub link: 
https://github.com/apache/kvrocks/discussions/3044#discussioncomment-13690655

----
This is an automatically sent email for [email protected].
To unsubscribe, please send an email to: [email protected]

Reply via email to