Skip to content

Conversation

rekby
Copy link
Member

@rekby rekby commented Mar 7, 2023

I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en

Pull request type

Please check the type of change your PR introduces:

  • Bugfix
  • [v] Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

What is the current behavior?

If server truncate result - sdk allow read returned result. It is client responsibility to

What is the new behavior?

If server sent truncated response - raise exception.
Allow truncated response possible with create table client with enable option with_allow_truncated_result.

@rekby rekby changed the base branch from main to v3 March 7, 2023 14:29
@rekby rekby closed this Mar 7, 2023
@rekby rekby reopened this Mar 7, 2023
@rekby rekby force-pushed the v3-error-on-truncated-result branch from 57c9253 to a123228 Compare March 7, 2023 14:36
@rekby rekby marked this pull request as ready for review March 7, 2023 16:39
@rekby rekby merged commit 95ab809 into v3 Mar 7, 2023
@rekby rekby deleted the v3-error-on-truncated-result branch March 7, 2023 16:40
@rekby rekby linked an issue Mar 7, 2023 that may be closed by this pull request
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.

error with truncated result by default for v3

1 participant