Skip to content

Fix link to timeout best practices #2729

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

Merged
merged 1 commit into from
Apr 11, 2025
Merged

Fix link to timeout best practices #2729

merged 1 commit into from
Apr 11, 2025

Conversation

marciw
Copy link
Contributor

@marciw marciw commented Apr 11, 2025

Preview (third link)

Missed this in #2727 -- update the link to timeout best practices now that it's back in this repo.

@marciw marciw marked this pull request as ready for review April 11, 2025 13:38
Copy link
Contributor

@colleenmcginnis colleenmcginnis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@marciw marciw enabled auto-merge (squash) April 11, 2025 13:43
@ezimuel ezimuel self-assigned this Apr 11, 2025
@ezimuel ezimuel requested review from ezimuel and removed request for JoshMock April 11, 2025 14:02
Copy link
Contributor

@ezimuel ezimuel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@marciw marciw closed this Apr 11, 2025
auto-merge was automatically disabled April 11, 2025 14:07

Pull request was closed

@marciw marciw reopened this Apr 11, 2025
@ezimuel ezimuel merged commit 8b4fcc8 into elastic:main Apr 11, 2025
18 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 11, 2025
ezimuel pushed a commit that referenced this pull request Apr 11, 2025
marciw added a commit to elastic/docs-content that referenced this pull request Apr 11, 2025
Troubleshooting content for the clients has moved back to the client
docs repos. This PR makes corresponding changes in docs-content.

Related PRs:
elastic/elasticsearch-java#988
elastic/elasticsearch-js#2727
elastic/elasticsearch-net#8490
elastic/elasticsearch-ruby#2628
elastic/elasticsearch-js#2729
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants