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

Adjust column name to match Spark job #118

Merged
merged 1 commit into from Aug 17, 2017
Merged

Adjust column name to match Spark job #118

merged 1 commit into from Aug 17, 2017

Conversation

@c-w
Copy link
Contributor

@c-w c-w commented Aug 17, 2017

Based on https://github.com/CatalystCode/project-fortis-pipeline/pull/117/files#diff-e79522b8c1a9d91a3f8005389e6302c2L168, to get this merged faster.

@c-w c-w requested a review from erikschlegel Aug 17, 2017
@c-w c-w mentioned this pull request Aug 17, 2017
@c-w
Copy link
Contributor Author

@c-w c-w commented Aug 17, 2017

Note that ideally the bigint columns in this table should be counters. Using counters it's not needed to read back the old values and one can just insert the increment. Counters are also cassandra native and therefore race conditions are handled under the hood.

c-w added a commit to CatalystCode/project-fortis-services that referenced this issue Aug 17, 2017
Copy link
Contributor

@erikschlegel erikschlegel left a comment

LGTM

@c-w c-w merged commit 56aee65 into master Aug 17, 2017
2 checks passed
@c-w c-w deleted the fix-computedtiles branch Aug 17, 2017
@c-w c-w removed the in progress label Aug 17, 2017
rachelnicole pushed a commit that referenced this issue Jan 24, 2018
… Settings Schema (#118)

* Fixed admin setting integration issues

* A variety of bug fixes to support pulling fortis events from the cassandra table into the news feed.

* A variety of fixes to support showing fortis events in the news feed.

* Fixing linting errors

* Reverting executeQuery change

* Fixing lint errors

* Messages byBbox schema changes round II

* Consolidating mainEdge and filteredEdges as conjunctivetopics field

* Addressing clemens PR review comments

* Restructuring edge schema, and moving admin related services to settings schema

* rebasing

* Fixing top sources schema

* Fixing lint errors

* Fixing lint errors
rachelnicole pushed a commit that referenced this issue Jan 24, 2018
…licy

Use same load balancing policy as fortis-services
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants