fix(source-mssql): Improve invalid CDC offset message#78297
fix(source-mssql): Improve invalid CDC offset message#78297devin-ai-integration[bot] wants to merge 3 commits into
Conversation
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
Detected
|
|
|
Deploy preview for airbyte-docs ready!
Deployed with vercel-action |
What
Resolves https://github.com/airbytehq/airbyte-internal-issues/issues/16455
Related to https://github.com/airbytehq/oncall/issues/12670
Improve the
source-mssqlCDC warm-start error message for saved offsets that are no longer valid for SQL Server CDC logs. The previous message included the concrete saved LSN value and generic reset instructions, which made the user-facing message non-deterministic and harder to aggregate.Requested by devin-ai-integration[bot] via the
/ai-triage-error-messageplaybook.How
source-mssqlfrom4.4.7to4.4.8and add a changelog entry linked to this PR.Review guide
airbyte-integrations/connectors/source-mssql/src/main/kotlin/io/airbyte/integrations/source/mssql/MsSqlServerDebeziumOperations.ktairbyte-integrations/connectors/source-mssql/metadata.yamldocs/integrations/sources/mssql.mdUser Impact
Users with invalid or aged-out SQL Server CDC offsets see a deterministic message that names the SQL Server CDC log range / retention-window cause instead of exposing an internal saved LSN value. Connector behavior and FailureType are unchanged.
Can this PR be safely reverted and rolled back?
Testing
/home/ubuntu/repos/airbyte/gradlew -p /home/ubuntu/repos/airbyte :airbyte-integrations:connectors:source-mssql:compileKotlin :airbyte-integrations:connectors:source-mssql:compileTestKotlin -x testpre-commit run --files airbyte-integrations/connectors/source-mssql/src/main/kotlin/io/airbyte/integrations/source/mssql/MsSqlServerDebeziumOperations.kt docs/integrations/sources/mssql.md airbyte-integrations/connectors/source-mssql/metadata.yaml(prettier and license passed; Spotless hook could not complete locally because Maven is not installed, so the ktfmt change was applied from CI output)Important
Active progressive rollout warning for source-mssql.
Devin session