Skip to content

DMD-92 Throw meaningful exception when view is out of sync #1486

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.

Sign up for GitHub

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

Jump to bottom
Merged
tomasfejfar merged 1 commit into from
Apr 23, 2025
Merged

DMD-92 Throw meaningful exception when view is out of sync #1486

tomasfejfar merged 1 commit into from
Apr 23, 2025

Conversation

@tomasfejfar
Copy link
Contributor

@tomasfejfar tomasfejfar commented Apr 22, 2025 •
edited
Loading

Jira: DMD-92
KBC: https://github.com/keboola/connection/pull/5685

Before asking for review make sure that:

Checklist

  • New client method(s) has tests
  • New client method(s) support branches, or they are listed in BranchAwareClient
  • Apiary file is updated

Release

  • I gave the PR a proper label:
    • major (BC break)
    • minor (new feature)
    • patch (backwards compatible fix)
    • no release (just test changes)

@tomasfejfar tomasfejfar added the no release just test changes label Apr 22, 2025
@tomasfejfar tomasfejfar force-pushed the tf-DMD-92-sds-change-of-datastructure-should-not-cause-internal-error branch 2 times, most recently from 411b69d to 93af5a9 Compare April 22, 2025 13:51
@tomasfejfar tomasfejfar force-pushed the tf-DMD-92-sds-change-of-datastructure-should-not-cause-internal-error branch from 93af5a9 to ee04ab2 Compare April 23, 2025 08:20
@tomasfejfar tomasfejfar force-pushed the tf-DMD-92-sds-change-of-datastructure-should-not-cause-internal-error branch from ee04ab2 to 6af4612 Compare April 23, 2025 08:56
@tomasfejfar tomasfejfar merged commit 51eda9c into master Apr 23, 2025
4 checks passed
@tomasfejfar tomasfejfar deleted the tf-DMD-92-sds-change-of-datastructure-should-not-cause-internal-error branch April 23, 2025 09:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vojtabiberle vojtabiberle vojtabiberle approved these changes

Assignees

No one assigned

Labels

no release just test changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tomasfejfar @vojtabiberle