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

fix: use dsn::tools::native_aio_provider as default value of FLAGS_aio_factory_name #774

Merged
merged 14 commits into from
Mar 8, 2021

Conversation

levy5307
Copy link
Contributor

@levy5307 levy5307 commented Mar 4, 2021

  1. In previous implementation, there are some code to control the value FLAGS_aio_factory_name. But it is a little complexity which can lead to problems. So in this pull request, I remove these code and use dsn::tools::native_aio_provider as default value of FLAGS_aio_factory_name. You can config FLAGS_aio_factory_name as dsn::tools::sim_aio_provider if you want to use sim_aio_provider

  2. fix the bug below:

➜  pegasus git:(master) ✗ ./run.sh shell
W2021-03-04 13:30:50.875 (1614835850875376939 16541) : overwrite default thread pool for task RPC_CM_QUERY_PARTITION_CONFIG_BY_INDEX from THREAD_POOL_META_SERVER to THREAD_POOL_DEFAULT
W2021-03-04 13:30:50.875 (1614835850875414106 16541) : 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.

cannot find factory '' with factory type provider
	the following 2 factories are registered:
		dsn::tools::native_aio_provider (type: provider)
		dsn::tools::sim_aio_provider (type: provider)
	Please specify the correct factory name in your tool_app or in configuration file
The config file is: /home/mi/workspace/pegasus/config-shell.ini.16534
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
>>>

@levy5307 levy5307 marked this pull request as draft March 4, 2021 08:30
@levy5307 levy5307 marked this pull request as ready for review March 5, 2021 02:54
@levy5307 levy5307 merged commit bd60a14 into XiaoMi:master Mar 8, 2021
@levy5307 levy5307 deleted the aio-provider-config branch March 8, 2021 02:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants