Skip to content
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

refactor: remove some useless configs #338

Merged
merged 10 commits into from
Jun 17, 2019
Merged

Conversation

neverchanje
Copy link
Contributor

@neverchanje neverchanje commented Jun 15, 2019

What problem does this PR solve?

The configurations

  • max_input_queue_length
  • hpc_logger
  • cli_local
  • cli_remote
  • updating_rocksdb_sstsize_interval_seconds
  • start_nfs
  • fast_execution_in_network_thread
  • fast_run

already have no codes related.

Previously @qinzuoyan had cleared max_input_queue_length from rdsn: XiaoMi/rdsn#191
@shengofsun has cleared cli_local and cli_remote from rdsn: XiaoMi/rdsn#156
@acelyc111 has changed updating_rocksdb_sstsize_interval_seconds to update_rdb_stat_interval #212
@shengofsun has cleared start_nfs from rdsn: XiaoMi/rdsn#142
@qinzuoyan has cleared fast_execution_in_network_thread from rdsn in 2016: XiaoMi/rdsn@2e43e94

fast_run was removed in XiaoMi/rdsn#117

This time I do the same thing in Pegasus.

What is changed and how it works?

Check List

Tests

  • No code

Related changes

  • Need to update the documentation

@neverchanje neverchanje changed the title refactor: remove max_input_queue_length and hpc_logger from configs refactor: remove some useless configs Jun 15, 2019
@qinzuoyan qinzuoyan merged commit 3c59ffd into apache:master Jun 17, 2019
@neverchanje neverchanje deleted the refactor branch June 17, 2019 08:46
@neverchanje neverchanje mentioned this pull request Jul 12, 2019
neverchanje pushed a commit to neverchanje/pegasus that referenced this pull request Jul 13, 2019
Former-commit-id: b05475bc0d7a15015c5317956bb444b40967f873 [formerly 3c59ffd]
Former-commit-id: 4c1096f215f71ac17d556f15df58939b6c6bfe8a
neverchanje pushed a commit that referenced this pull request Jul 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants