-
Notifications
You must be signed in to change notification settings - Fork 25
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
Exception message not printed out if it's not json in runTransform #61
Exception message not printed out if it's not json in runTransform #61
Conversation
/gcbrun |
ingestion/function/index.js
Outdated
await dataset.createTable(config.destinationTable, { schema: config.metadata.fields }); | ||
} | ||
const transform = `SELECT ${transformQuery}, '${batchId}' AS ${processPrefix}_batch_id FROM \`${config.dataset}.${config.stagingTable}\``; | ||
// const exists = tableExists(config.dataset, config.destinationTable); |
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.
For now, commenting out the exists check as it was broken. As such there is no change in behavior here. Once we address the mode/description issue we can add this back as needed.
@@ -213,8 +214,10 @@ async function fromStorage(bucket, file) { | |||
async function tableExists(datasetId, tableName) { |
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.
Fixed issue where this function wasn't returning a boolean.
LGTM |
Fixes: