-
Notifications
You must be signed in to change notification settings - Fork 0
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
conformance-profiles: Database table and worker to process data #78
Conversation
3de9465
to
24190ea
Compare
'ON normalised_data_profile_results.normalised_data_id = normalised_data.id AND normalised_data_profile_results.profile_name=$1 '+ | ||
'WHERE normalised_data_profile_results.normalised_data_id IS NULL AND normalised_data.data_deleted=FALSE '+ | ||
'ORDER BY normalised_data.updated_at ASC LIMIT 10'; | ||
|
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.
You can use back ticks ` for multi line strings rather than concatenation at run time.
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 ta
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 good to update the list of tools to the README too if you can. No feedback blocking approve here though.
'ORDER BY normalised_data.updated_at ASC LIMIT 10'; | ||
|
||
while(true) { | ||
|
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 usually try to make sure there is some kind of safety catch in a while true, might be worth adding if there is way to do that
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.
Point - The catch() block should have break in it - that will apply in several places tho so I'll do that in another P.R.
I'll merge as this blocks some other things then do the readme straight away - forgot that, sorry. Thanks. |
#33