-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Bump mssql connector #14649
Bump mssql connector #14649
Conversation
/test connector=connectors/destination-mssql
Build PassedTest summary info:
|
@benmoriceau have you tested this upgrade scenario:
|
@@ -115,6 +115,7 @@ Using this feature requires additional configuration, when creating the source. | |||
|
|||
| Version | Date | Pull Request | Subject | | |||
|:--------| :--- |:---------------------------------------------------------|:----------------------------------------------------------------------------------------------------| | |||
| 0.1.20 | 2022-07-12 | [14649](https://github.com/airbytehq/airbyte/pull/14649) | Include lifecycle management | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
State Lifecycle management?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It refers to #13670.
No
@grishick No, I'm gonna try to do it. |
@grishick I tested it locally and it worked, I'll publish once I have your approval. |
What
Bump the connector version to make sure that the state lifecycle manager is included.