-
Notifications
You must be signed in to change notification settings - Fork 80
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
Data objects #75
Data objects #75
Conversation
…submitted_to_insdc methods, as well as adding their tests
This is ready for review/merge!!! Any takers? |
"INSERT INTO qiita.{0} (filepath, filepath_type_id, checksum, " | ||
"checksum_algorithm_id) VALUES {1} " | ||
"RETURNING filepath_id".format(self._filepath_table, | ||
', '.join(values))) |
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.
would be safer here to just use %s instead of direct values and add the values in using psycopg2.
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.
Ah, I see, weird execute many for fetchall. Still would be good to use one of the escape functions to clean the values before entering them into the db.
Just a couple comments. Let me know when I can do another review. |
@squirrelo Ready for a new review. |
1 similar comment
👍 |
Fixes #54
Fixes #48
Fixes #47
Fixes #46
Introduces the raw/preprocessed/processed data objects. As discussed offline with @adamrp, since the database is aware of filepaths