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

[Feature] Deprecate the logic sharding concept in the OAP #10786

Open
2 of 3 tasks
wu-sheng opened this issue May 10, 2023 · 3 comments
Open
2 of 3 tasks

[Feature] Deprecate the logic sharding concept in the OAP #10786

wu-sheng opened this issue May 10, 2023 · 3 comments
Assignees
Labels
backend OAP backend related. feature New feature

Comments

@wu-sheng
Copy link
Member

Search before asking

  • I had searched in the issues and found no similar feature requirement.

Description

    # Enable shard metrics and records indices into multi-physical indices, one index template per metric/meter aggregation function or record.
    logicSharding: ${SW_STORAGE_ES_LOGIC_SHARDING:false}

Currently, we have logicSharding concept in the OAP's booting parameter list. This could be manually controlled by the users. But still, I feel this is a little exposing too much, as actually, this is an expired feature already.

I want to propose an automatic sharding mechanism to replace this.

  1. OAP always booting with no-logic-sharding in the clean installation.
  2. OAP could automatically pick up the logic sharding value when the configuration is absent or logicSharding=false.
  3. Flag logicSharding setting deprecated as we would never recommend the user to set it manually again.
  4. We remove this configuration from YAML and only keep the system env reading(SW_STORAGE_ES_LOGIC_SHARDING) in the code in case of force setting to resolve bugs in urgent.

Use case

As a result of this enhancement, we could always have the latest and recommended index pattern in the new installation, and would not break users' old deployment.

Related issues

No response

Are you willing to submit a PR?

  • Yes I am willing to submit a PR!

Code of Conduct

@wu-sheng wu-sheng added backend OAP backend related. feature New feature labels May 10, 2023
@wu-sheng wu-sheng added this to the 9.5.0 milestone May 10, 2023
@wu-sheng
Copy link
Member Author

@wankai123 Could you take a look and provide feedback about this proposal?

@wankai123
Copy link
Member

Make sense. As the merge mode has been set by default since 9.2.0. I think it is time to remove it from the YAML and only keep it in system env in case the old env doesn't want to wipe the storage when upgrading.

@wu-sheng
Copy link
Member Author

Another option is we permanently remove this feature in a milestone. Don't have to be 9.5.0. I am considering two iterations(9.7.0) later.

@wu-sheng wu-sheng removed this from the 9.5.0 milestone May 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend OAP backend related. feature New feature
Projects
None yet
Development

No branches or pull requests

3 participants