Skip to content
This repository has been archived by the owner. It is now read-only.

Add endpoint to query for facts #163

Merged
merged 3 commits into from Nov 28, 2017

Conversation

Projects
None yet
2 participants
@c-w
Copy link
Member

commented Oct 16, 2017

This is the service-layer change for this database change: CatalystCode/project-fortis-pipeline#165

image

Also fix two potential null references

@c-w c-w requested review from erikschlegel and jcjimenez and removed request for erikschlegel Oct 16, 2017

c-w added some commits Oct 16, 2017

@c-w c-w force-pushed the query-facts-view branch from 53e5e74 to 99c6d00 Oct 31, 2017

@c-w c-w force-pushed the query-facts-view branch from 99c6d00 to b35ff4f Oct 31, 2017

const pipelineQuery = `
SELECT eventid
FROM fortis.eventsbypipeline
WHERE pipelinekey IN ?

This comment has been minimized.

Copy link
@erikschlegel

erikschlegel Nov 27, 2017

Collaborator

Assuming this query will change once the materialized view is altered.

@c-w c-w referenced this pull request Nov 27, 2017

Closed

No Facts Loading #220

@c-w c-w merged commit aaf88f4 into master Nov 28, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@c-w c-w deleted the query-facts-view branch Nov 28, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.