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

[Search Pipelines] Remove Experimental Flag #8489

Closed
macohen opened this issue Jul 6, 2023 · 2 comments
Closed

[Search Pipelines] Remove Experimental Flag #8489

macohen opened this issue Jul 6, 2023 · 2 comments
Assignees
Labels
Search Search query, autocomplete ...etc v2.9.0 'Issues and PRs related to version v2.9.0'

Comments

@macohen
Copy link
Contributor

macohen commented Jul 6, 2023

I don't think this is the typical process, but I'd like to suggest that in the 2.9 release we update https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/common/util/FeatureFlags.java instead of just removing the feature flag itself. So, we would update:

public static final Setting<Boolean> SEARCH_PIPELINE_SETTING = Setting.boolSetting(SEARCH_PIPELINE, false, Property.NodeScope);

to

public static final Setting<Boolean> SEARCH_PIPELINE_SETTING = Setting.boolSetting(SEARCH_PIPELINE, true, Property.NodeScope);

The advantage is that we minimize change in the first release and then remove the flag in 2.10. Of course, I may not understand how the feature flags work in the first place and we can only remove the flag instead of setting it to true first.

@macohen
Copy link
Contributor Author

macohen commented Jul 6, 2023

cc: @mingshl, @noCharger, @sejli

@macohen macohen added v2.9.0 'Issues and PRs related to version v2.9.0' Search Search query, autocomplete ...etc and removed untriaged labels Jul 6, 2023
@noCharger noCharger self-assigned this Jul 6, 2023
@noCharger
Copy link
Contributor

noCharger commented Jul 11, 2023

The AI to fully remove the feature flag:

  • Keep the setting, but make it true by default. (Done v2.9.0)
  • Backport that to 2.x (Done v2.9.0)
  • Stop enabling the setting for mixed cluster tests (because it is enabled on main and 2.x). (Target next release)
  • Remove the setting.(Target next release)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Search Search query, autocomplete ...etc v2.9.0 'Issues and PRs related to version v2.9.0'
Projects
Status: Done
Development

No branches or pull requests

2 participants