reduce interrupt and shutdown delays to 1 minutes and 2 minutes when stopping a connector (initially set at 60minutes and 70minutes) - #35527
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. Join Stephane Geneix (@stephane-airbyte) and the rest of your teammates on |
Stephane Geneix (stephane-airbyte)
force-pushed
the
stephane/02-21-reduce_interrupt_and_shutdown_delays_to_1_minutes_and_2_minutes_when_stopping_a_connector_initially_set_at_60minutes_and_70minutes_
branch
from
February 22, 2024 23:59
e0cc6f5 to
3c29d6c
Compare
Stephane Geneix (stephane-airbyte)
requested a review
from a team
as a code owner
February 22, 2024 23:59
Akash Kulkarni (akashkulk)
approved these changes
Feb 23, 2024
Akash Kulkarni (akashkulk)
left a comment
Contributor
There was a problem hiding this comment.
Good find!
- Can you link the slack thread where we discussed this (for posterity + context)
- Are updates to other certified connectors being done in follow-ups?
Approving to unblock
Stephane Geneix (stephane-airbyte)
force-pushed
the
stephane/02-21-reduce_interrupt_and_shutdown_delays_to_1_minutes_and_2_minutes_when_stopping_a_connector_initially_set_at_60minutes_and_70minutes_
branch
from
February 23, 2024 00:14
3c29d6c to
6d41c28
Compare
Contributor
Author
|
/publish-java-cdk
|
…stopping a connector (initially set at 60minutes and 70minutes)
Stephane Geneix (stephane-airbyte)
force-pushed
the
stephane/02-21-reduce_interrupt_and_shutdown_delays_to_1_minutes_and_2_minutes_when_stopping_a_connector_initially_set_at_60minutes_and_70minutes_
branch
from
February 23, 2024 16:49
6d41c28 to
4ef7915
Compare
…wn_delays_to_1_minutes_and_2_minutes_when_stopping_a_connector_initially_set_at_60minutes_and_70minutes_
Contributor
Author
Merge activity
|
Stephane Geneix (stephane-airbyte)
deleted the
stephane/02-21-reduce_interrupt_and_shutdown_delays_to_1_minutes_and_2_minutes_when_stopping_a_connector_initially_set_at_60minutes_and_70minutes_
branch
February 23, 2024 22:12
Tanawat Panmongkol (tanawatpan)
pushed a commit
to tanawatpan/airbyte
that referenced
this pull request
Feb 24, 2024
…stopping a connector (initially set at 60minutes and 70minutes) (airbytehq#35527) Fixes airbytehq#32348 discussed here : https://airbytehq-team.slack.com/archives/C02U2SSHP9S/p1708552465201999
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Fixes #32348
discussed here : https://airbytehq-team.slack.com/archives/C02U2SSHP9S/p1708552465201999