Added function insertBatch to insert multiple values#21
Conversation
Update simplemysq.py added SSL Connectivity
Now insert values in batches
removed unnecessary lines
|
At first glance this looks good overall. Could you please clean up the merge conflict on the README? Also, and I'm not sure if this is me being nitpicky, but adding your name to the credit header in simplemysql.py seems unnecessary. I have write access on this repo and my name isn't in there, and neither is the name of any other contributors. I'm open to discussion on this, as I'm not sure what the de-facto standard is for this situation. |
Zetaphor
left a comment
Zetaphor
left a comment
There was a problem hiding this comment.
Please update your copy so your changes are added to the latest revision of this file
Zetaphor
left a comment
Zetaphor
left a comment
There was a problem hiding this comment.
Please update your copy so your changes are added to the latest revision of this file
|
Ahh I am sorry, that was my local copy which i uploaded, i keep track of files with my name, i will understand for which i have contributed. |
Removed name
|
Can some one please merge this issue. |
|
Merge complete, thank you for this contribution! Next time please make sure your fork is up to date with master so we don't keep having to resolve these README conflicts ;-) |
2 participants
Added function insertBatch to insert multiple values