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

[DOC] Concurrent Segment Search support in OpenSearch #3662

Closed
1 of 4 tasks
sohami opened this issue Apr 4, 2023 · 16 comments · Fixed by #6356
Closed
1 of 4 tasks

[DOC] Concurrent Segment Search support in OpenSearch #3662

sohami opened this issue Apr 4, 2023 · 16 comments · Fixed by #6356
Assignees
Labels
3 - Done Issue is done/complete v2.12.0
Milestone

Comments

@sohami
Copy link
Contributor

sohami commented Apr 4, 2023

What do you want to do?

  • Request a change to existing documentation
  • Add new documentation
  • Report a technical problem with the documentation
  • Other

Tell us about your request. Provide a summary of the request and all versions that are affected.
Add documentation about the concurrent segment search functionality and the knobs it exposes for the users to consume. For now adding as a placeholder issue and details will be added later.

What other resources are available? Provide links to related issues, POCs, steps for testing, etc.
opensearch-project/OpenSearch#2587

** Additional Issues **
#4281

@Naarcha-AWS
Copy link
Collaborator

@sohami: Which version is concurrent segment search going into?

@Naarcha-AWS Naarcha-AWS added 1 - Backlog Issue: The issue is unassigned or assigned but not started v-All This issue is valid for all versions through 1.3 and removed untriaged labels Apr 24, 2023
@jed326
Copy link
Contributor

jed326 commented Jun 12, 2023

Taking a note to mention the settings from opensearch-project/OpenSearch#7956. Specifically, the index setting for concurrent search will override the cluster setting, which I believe is a new behavior for settings in general.

@sohami
Copy link
Contributor Author

sohami commented Jun 12, 2023

@sohami: Which version is concurrent segment search going into?

@Naarcha-AWS For now we are targeting for 2.10

@jed326
Copy link
Contributor

jed326 commented Jul 10, 2023

As a part of opensearch-project/OpenSearch#8371 we are updating the definition of terminate_after search parameter to be per leaf slice instead of per shard. In the non-concurrent search case there is no behavior change.

@jed326
Copy link
Contributor

jed326 commented Aug 28, 2023

As a part of opensearch-project/OpenSearch#9469 there is a new supportsConcurrentSegmentSearch method in the AggregatorFactory class that needs to be overridden for any new aggregators that intend to support concurrent segment search.

@kolchfa-aws
Copy link
Collaborator

When this feature is GA, we should incorporate the additional introduced/modified fields in their respective tables in the API pages. See #4990 for the changed APIs.

@kolchfa-aws
Copy link
Collaborator

@sohami Is there any documentation needed for the 2.11 version for this issue?

@kolchfa-aws
Copy link
Collaborator

Per offline sync with @sohami, there's no work to be done for 2.11 for this issue

@kolchfa-aws kolchfa-aws modified the milestones: v2.10, v2.12 Oct 3, 2023
@sohami
Copy link
Contributor Author

sohami commented Oct 3, 2023

Yes we created separate doc issue for 2.11 where specific updates are needed. Thanks for moving this to 2.12 for now.

@kolchfa-aws
Copy link
Collaborator

@sohami Is there any documentation needed for the 2.12 version for this issue? Or is all the work done in #4281? Thanks!

@jed326
Copy link
Contributor

jed326 commented Dec 12, 2023

Taking note to document "slice_size" change from opensearch-project/OpenSearch#11585

@hdhalter hdhalter added 1 - Backlog - DEV Developer assigned to issue is responsible for creating PR. and removed 1 - Backlog Issue: The issue is unassigned or assigned but not started labels Jan 8, 2024
@jed326
Copy link
Contributor

jed326 commented Jan 30, 2024

@kolchfa-aws Compiling a list of all the docs related updates we will need for concurrent segment search GA:

@sohami @ticheng-aws @yigithub @neetikasinghal please let me know if you think of anything else!

@ticheng-aws
Copy link
Contributor

@kolchfa-aws Compiling a list of all the docs related updates we will need for concurrent segment search GA:

@sohami @ticheng-aws @yigithub @neetikasinghal please let me know if you think of anything else!

We also need a PR to remove the "experimental" keyword altogether.

@jed326
Copy link
Contributor

jed326 commented Feb 5, 2024

Added terms aggregation concurrent search docs and improvements here: #6355

@hdhalter hdhalter added 2 - In progress Issue/PR: The issue or PR is in progress. and removed 1 - Backlog - DEV Developer assigned to issue is responsible for creating PR. labels Feb 6, 2024
@jed326
Copy link
Contributor

jed326 commented Feb 6, 2024

@github-project-automation github-project-automation bot moved this from Todo to Done in Concurrent Search Feb 12, 2024
@hdhalter hdhalter added 3 - Done Issue is done/complete and removed 2 - In progress Issue/PR: The issue or PR is in progress. labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Done Issue is done/complete v2.12.0
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

6 participants