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

"copy INSERT statement" should have line-breaks between rows #104

Closed
qwesda opened this Issue Oct 29, 2015 · 4 comments

Comments

Projects
None yet
2 participants
@qwesda

qwesda commented Oct 29, 2015

copying many rows with the "copy INSERT statement" leads potentially to one very long line.
Sadly many text editors handle long lines very badly, especially if they try to apply syntax highlighting.
Splitting the statement over many lines removes this issue and also improves the readability of the copied statement.

@jakob jakob added the started label Oct 29, 2015

@jakob

This comment has been minimized.

Show comment
Hide comment
@jakob
Owner

jakob commented Oct 29, 2015

@qwesda

This comment has been minimized.

Show comment
Hide comment
@qwesda

qwesda Oct 29, 2015

works as expected and doesn't kill sublime text anymore (with syntax highlighting enabled)! Tested with 5k rows and 1.2MB total statement length. Anything bigger should probably not have been a copy statement operation anyway.

I would add two additional line breaks: one before and one after VALUES

qwesda commented Oct 29, 2015

works as expected and doesn't kill sublime text anymore (with syntax highlighting enabled)! Tested with 5k rows and 1.2MB total statement length. Anything bigger should probably not have been a copy statement operation anyway.

I would add two additional line breaks: one before and one after VALUES

@jakob

This comment has been minimized.

Show comment
Hide comment
@jakob

jakob Nov 2, 2015

Owner

I've added the two additional newlines you suggested. Good idea.
https://eggerapps-downloads.s3.amazonaws.com/postico-1240.zip

Owner

jakob commented Nov 2, 2015

I've added the two additional newlines you suggested. Good idea.
https://eggerapps-downloads.s3.amazonaws.com/postico-1240.zip

@qwesda

This comment has been minimized.

Show comment
Hide comment
@qwesda

qwesda Nov 3, 2015

works great ...
it's also nice to see the indices even they are not yet editable.

qwesda commented Nov 3, 2015

works great ...
it's also nice to see the indices even they are not yet editable.

@qwesda qwesda closed this Nov 3, 2015

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