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

[BUG] BulkAlias summary saying it calls indices.stats #715

Closed
jumc opened this issue Jul 17, 2024 · 3 comments
Closed

[BUG] BulkAlias summary saying it calls indices.stats #715

jumc opened this issue Jul 17, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@jumc
Copy link

jumc commented Jul 17, 2024

What is the bug?

Method summary informs wrong API call.

How can one reproduce the bug?

Look at BulkAlias(Func<BulkAliasDescriptor, IBulkAliasRequest> selector) docs through an IDE

What is the expected behavior?

/// <summary>
/// <c>POST</c> request to the <c>indices.update_aliases</c> API, read more about this API online:
/// <para></para>
/// <a href = "https://opensearch.org/docs/latest/opensearch/rest-api/alias/">https://opensearch.org/docs/latest/opensearch/rest-api/alias/</a>
/// </summary>
public BulkAliasResponse BulkAlias(Func<BulkAliasDescriptor, IBulkAliasRequest> selector) => BulkAlias(selector.InvokeOrDefault(new BulkAliasDescriptor()));		

Do you have any screenshots?

image

@dblock
Copy link
Member

dblock commented Jul 17, 2024

Thanks @jumc, want to fix it?

@jumc
Copy link
Author

jumc commented Jul 17, 2024

Sure! Here's the PR: #717
Since this is my first time contributing, please let me know if I'm missing something

@Xtansia Xtansia removed the untriaged label Jul 18, 2024
@dblock
Copy link
Member

dblock commented Oct 30, 2024

Closed via #717.

@dblock dblock closed this as completed Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants