Skip to content
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

Server exceptions are buried by ClickHouseBulkCopy #321

Closed
smbecker opened this issue Jun 30, 2023 · 1 comment
Closed

Server exceptions are buried by ClickHouseBulkCopy #321

smbecker opened this issue Jun 30, 2023 · 1 comment
Assignees
@DarkWanderer
Labels
bug Something isn't working

Comments

@smbecker
Copy link
Contributor

I was having issues where it seemed that ClickHouseBulkCopy.WriteToServerAsync was completing successfully and providing the number of rows written. However, the table in CH was still empty. I found that this continuation blindly ignores the exception that it thrown from PostStreamAsync.
@DarkWanderer DarkWanderer self-assigned this Jul 1, 2023
@DarkWanderer DarkWanderer added the bug Something isn't working label Jul 1, 2023
DarkWanderer added a commit that referenced this issue Jul 1, 2023
@DarkWanderer
Fixed exception handling in BulkCopy #321
481062a
DarkWanderer added a commit that referenced this issue Jul 1, 2023
@DarkWanderer
Fixed exception handling in BulkCopy #321 (#322)
4a78793
@DarkWanderer
Copy link
Owner

Thank you for report, this is indeed a very annoying issue. Fix will be available in version 6.7.1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@DarkWanderer DarkWanderer

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@smbecker @DarkWanderer