Skip to content
This repository has been archived by the owner on Jun 23, 2022. It is now read-only.

feat: add env check entry in app_env_validator for user specified compaction #849

Merged
merged 1 commit into from
Jul 2, 2021

Conversation

levy5307
Copy link
Contributor

@levy5307 levy5307 commented Jul 2, 2021

Manual Test

➜  pegasus git:(master) ✗ ./run.sh shell
W2021-07-02 11:48:40.629 (1625197720629655537 28820) : overwrite default thread pool for task RPC_CM_QUERY_PARTITION_CONFIG_BY_INDEX from THREAD_POOL_META_SERVER to THREAD_POOL_DEFAULT
W2021-07-02 11:48:40.629 (1625197720629708604 28820) : overwrite default thread pool for task RPC_CM_QUERY_PARTITION_CONFIG_BY_INDEX_ACK from THREAD_POOL_META_SERVER to THREAD_POOL_DEFAULT
Pegasus Shell 2.1.SNAPSHOT
Type "help" for more information.
Type "Ctrl-D" or "Ctrl-C" to exit the shell.

The config file is: /home/mi/workspace/pegasus/config-shell.ini.28813
The cluster name is: onebox
The cluster meta list is: 127.0.0.1:34601,127.0.0.1:34602,127.0.0.1:34603
>>> ls
[general_info]
app_id  status     app_name  app_type  partition_count  replica_count  is_stateful  create_time          drop_time  drop_expire  envs_count  
1       AVAILABLE  stat      pegasus   4                3              true         2021-07-02_11:48:39  -          -            0           
2       AVAILABLE  temp      pegasus   8                3              true         2021-07-02_11:48:39  -          -            0           

[summary]
total_app_count  : 2

>>> use test
OK
>>> use temp
OK
>>> ls
[general_info]
app_id  status     app_name  app_type  partition_count  replica_count  is_stateful  create_time          drop_time  drop_expire  envs_count  
1       AVAILABLE  stat      pegasus   4                3              true         2021-07-02_11:48:39  -          -            0           
2       AVAILABLE  temp      pegasus   8                3              true         2021-07-02_11:48:39  -          -            0           

[summary]
total_app_count  : 2

>>> set_app_envs user_specified_compaction aaa
set app envs succeed
>>> 

@Smityz
Copy link
Contributor

Smityz commented Jul 2, 2021

LGTM

@hycdong hycdong merged commit fc224a4 into XiaoMi:master Jul 2, 2021
@hycdong hycdong added the type/app-envs This PR updates app_envs label Aug 31, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
type/app-envs This PR updates app_envs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants