You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The /{index}/_recovery API returns the following response.
The field names isSearchableSnapshot, remoteStoreIndexShallowCopy, sourceRemoteStoreRepository and sourceRemoteTranslogRepository are in non-standard format(using camelCase).
Thanks for filing this issue . We would welcome any contribution to correct this as per convention in 3.0 . Note that we wouldn't be backport this to 2.x as it would be a non backward compatible change .
Going forward , we should adhere to maintain the standard format in our APIs .
@linuxpi had a suggestion to add a gradle check to determine that we adhere to the standards in our api .
Hi team, I’ve submitted a PR that addresses issue #16334 by updating the field names in the /index/_recovery API response to use snake_case, aligning with the API’s naming conventions.
Describe the bug
The
/{index}/_recovery
API returns the following response.The field names isSearchableSnapshot, remoteStoreIndexShallowCopy, sourceRemoteStoreRepository and sourceRemoteTranslogRepository are in non-standard format(using camelCase).
Source code is also implemented in CamelCase.
https://github.com/opensearch-project/OpenSearch/blob/main/server/src/main/java/org/opensearch/cluster/routing/RecoverySource.java#L413-L421
Related component
Other
To Reproduce
Expected behavior
API must be returned in standard format.
Additional Details
Plugins
None
Screenshots
None
Host/Environment (please complete the following information):
Additional context
opensearch-project/opensearch-api-specification#611 (comment)
The text was updated successfully, but these errors were encountered: