-
Notifications
You must be signed in to change notification settings - Fork 965
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enable iotdb to set useasyncserver and useasyncapplier through configuration files #1588
Conversation
is_use_async_server=true | ||
|
||
#is use sync applier | ||
is_use_async_applier=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please make the annotation more clearer.
config.setUseAsyncServer( | ||
Boolean.parseBoolean(properties.getProperty("is_use_async_server", | ||
String.valueOf(config.isUseAsyncServer())))); | ||
logger.info("server+{}",config.isUseAsyncServer()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove the debug log
config.setUseAsyncApplier( | ||
Boolean.parseBoolean(properties.getProperty("is_use_async_applier", | ||
String.valueOf(config.isUseAsyncApplier())))); | ||
logger.info("app+{}",config.isUseAsyncApplier()); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the same as above
is_use_async_server=true | ||
|
||
#is use sync applier | ||
is_use_async_applier=true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
the default value is false. make sure consistent with the code
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your contribution.
Actually we abstract out these two parameters as the experimental variables to test the performance. We would choose the configuration with the best performance when the distributed version is released, so we do not need to add them to the configuration item at present.
It is convenient to change the corresponding variables in the code by modifying the configuration file during testing. It is not necessary to change the code and recompile before each test.