-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
dbeaver/dbeaver#26613 Fix json format for binary column exporting #34059
dbeaver/dbeaver#26613 Fix json format for binary column exporting #34059
Conversation
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.
I don't like the general approach. Let's discuss it tomorrow.
@@ -53,6 +53,8 @@ public interface IStreamDataExporterSite { | |||
|
|||
void writeBinaryData(@NotNull DBDContentStorage cs) throws IOException; | |||
|
|||
void writeBinaryData(@NotNull DBDContentStorage cs, String typeOfOutput) throws IOException; |
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.
Annotations 🙁
writer.write(binaryString); | ||
break; | ||
} | ||
} | ||
default: { | ||
// Binary stream | ||
try (Reader reader = new InputStreamReader(stream, cs.getCharset())) { | ||
String string = IOUtils.readToString(reader); |
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.
This is a big no. It might be a really big blob (like several GBs in size) that we would otherwise just pipe to the output stream (writer).
c2499e1
to
3327999
Compare
3327999
to
af53c81
Compare
e1f0f22
to
ed89583
Compare
verified |
No description provided.