opensearch-project / opensearch-net

OpenSearch .NET Client
Apache License 2.0
102 stars 49 forks source link

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

Open jumc opened 1 month ago

jumc commented 1 month ago

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 commented 1 month ago

Thanks @jumc, want to fix it?

jumc commented 1 month ago

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