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

Fix popularplaces table for edges queries #110

Merged
merged 2 commits into from Aug 16, 2017

Conversation

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

commented Aug 15, 2017

We query the popularplaces table like this:

    SELECT placeid, mentioncount, centroidlat, centroidlon
    FROM fortis.popularplaces
    WHERE period = ?
    AND periodtype = ?
    AND pipelinekey = ?
    AND externalsourceid = ?
    AND conjunctiontopic1 = ?
    AND conjunctiontopic2 = ?
    AND conjunctiontopic3 = ?
    AND (periodstartdate, periodenddate) <= (?, ?)
    AND (periodstartdate, periodenddate) >= (?, ?)

So we need to jiggle around some of the column keys so that we can query on period{start,end}date.

@c-w c-w requested a review from erikschlegel Aug 15, 2017

@c-w c-w added the in progress label Aug 15, 2017

Remove placename
We'll dereference it via placeid in the graphql layer
@jcjimenez
Copy link
Contributor

left a comment

LGTM

@c-w c-w merged commit c76c153 into master Aug 16, 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 fix-edges-queries branch Aug 16, 2017

@c-w c-w removed the in progress label Aug 16, 2017

rachelnicole pushed a commit that referenced this pull request Jan 24, 2018

Merge pull request #110 from CatalystCode/fix-sentiment
Fix sentiment dereference for latest model changes

rachelnicole pushed a commit that referenced this pull request Jan 24, 2018

Merge pull request #110 from CatalystCode/more-logging
Add more logging to cassandra sink
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.