Skip to content

Conversation

@shirshanka
Copy link
Contributor

@shirshanka shirshanka commented Aug 13, 2021

After the latest release, there are numerous reports that ingestion is timing out for folks due to latencies in environments.
Increasing the default timeout to 30 seconds to avoid unnecessary failures, while still being robust to complete outages.

Checklist

  • The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
  • Links to related issues (if applicable)
  • Tests for the changes have been added/updated (if applicable)
  • Docs related to the changes have been added/updated (if applicable)

@shirshanka shirshanka requested a review from woodyzhou August 13, 2021 17:13
Copy link
Contributor

@woodyzhou woodyzhou left a comment

Choose a reason for hiding this comment

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

LGTM

@shirshanka shirshanka merged commit 39a0081 into master Aug 13, 2021
@shirshanka shirshanka deleted the increase-emitter-timeout branch August 13, 2021 18:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants