LinkinStars commented on issue #1208:
URL: 
https://github.com/apache/incubator-answer/issues/1208#issuecomment-2560925717

   Very nice feature design. Let's add some more details.
   
   1. Let's add details on how `status` is used. Is it for deletion? Or is it 
for other identification purposes?
   2. `Tx` is a good design, but this one seems to be more difficult to 
implement. I don't have a good idea of how to implement it elegantly. Perhaps 
you might describe it more. I would consider this feature an enhancement, not a 
required item.
   3. Do you think we need to add a group concept and range query? As the 
plugin's data may hold data under different features. Under the current design, 
we can differentiate by key.
   
   key=value
   
   ```
   login_abc(userid)=123
   feat_ced(userid)=456
   ```
   
   Importing the concept of groups is actually
   
   ```
   key=abc; group=login; value=123
   key=ced; group=feat; value=456
   ```
   
   In this way, perhaps we can give the plugin the ability to page through the 
data under a group.
   
   Of course I would consider this an enhancement as well. Maybe for cases 
where the plugin has a scheduled task to scan all the existing data for 
processing.


-- 
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