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

Add placeids column #25

Merged
merged 1 commit into from Jun 28, 2017

Conversation

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

commented Jun 27, 2017

As noted in project-fortis-spark#17, this column is required to implement the popularLocations query in the project-fortis-services EdgesSchema.

Basically, to implement the query, we'll first fetch the geofence for the current site and then query the computedtiles table for those tiles inside of the geofence and then return the union of places contained in all the tiles.

Add placeids column
This is required to implement the popularLocations query in the
project-fortis-services EdgesSchema.

Basically, to implement the query, we'll first fetch the geofence for
the current site and then query the computedtiles table for those tiles
inside of the geofence and then return the union of places contained in
all the tiles.

@c-w c-w requested a review from erikschlegel Jun 27, 2017

@c-w c-w added the in progress label Jun 27, 2017

@erikschlegel
Copy link
Contributor

left a comment

LGTM

@c-w c-w merged commit 74f9d18 into master Jun 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 add-placeids-column branch Jun 28, 2017

@c-w c-w removed the in progress label Jun 28, 2017

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

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

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

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.