You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@pegasus.apache.org by GitBox <gi...@apache.org> on 2021/09/16 04:01:05 UTC

[GitHub] [incubator-pegasus] levy5307 opened a new pull request #814: fix: remove user specified compaction if the corresponding appenv was removed

levy5307 opened a new pull request #814:
URL: https://github.com/apache/incubator-pegasus/pull/814


   ### What problem does this PR solve? <!--add issue link with summary if exists-->
   1. add some logs
   2. remove user specified compaction if the corresponding appenv was removed
   3. make user specified compaction applicable to default_ttl 
   
   
   ### Checklist <!--REMOVE the items that are not applicable-->
   
   ##### Tests <!-- At least one of them must be included. -->
   
   - Manual test (add detailed scripts or steps below)
   
   ### remove user specified compaction if the corresponding appenv was removed
   1. insert data into app 'temp'
   ```
   >>> set hashkey sortkey value
   OK
   
   app_id          : 2
   partition_index : 2
   decree          : 36
   server          : 10.231.57.100:34803
   >>> get hashkey sortkey
   "value"
   
   app_id          : 2
   partition_index : 2
   server          : 10.231.57.100:34803
   ```
   2. add user specified compaction 
   ```
   Pegasus-AdminCli-1.1.0 » add-compaction-operation -o delete --hashkey-pattern "hashkey" --hashkey-match "perfix"
   ```
   3. start manual compaction
   ```
   ➜  pegasus git:(fix-compaction) ✗ ./scripts/pegasus_manual_compact.sh -c 127.0.0.1:34601,127.0.0.1:34602 -a temp
   ```
   4. get data from app 'temp', and we can see the data was deleted
   ```
   >>> use temp
   OK
   >>> get hashkey sortkey
   Not found
   
   app_id          : 2
   partition_index : 2
   server          : 10.231.57.100:34803
   ```
   5. insert data into app 'temp' again
   6. del appenv for user specified compaction
   ```
   >>> del_app_envs user_specified_compaction
   del app envs succeed
   =============================
   deleted keys:
       user_specified_compaction
   =============================
   ```
   7. start manual compaction again
   8. get data from app 'temp', and we can see the data is not deleted
   ```
   >>> get hashkey sortkey
   "value"
   
   app_id          : 2
   partition_index : 2
   server          : 10.231.57.100:34803
   ```
   


-- 
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: dev-unsubscribe@pegasus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org


[GitHub] [incubator-pegasus] levy5307 merged pull request #814: fix: remove user specified compaction if the corresponding appenv was removed

Posted by GitBox <gi...@apache.org>.
levy5307 merged pull request #814:
URL: https://github.com/apache/incubator-pegasus/pull/814


   


-- 
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: dev-unsubscribe@pegasus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@pegasus.apache.org
For additional commands, e-mail: dev-help@pegasus.apache.org