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

Ensure that eventid is unique #94

Merged
merged 1 commit into from Aug 16, 2017
Merged

Ensure that eventid is unique #94

merged 1 commit into from Aug 16, 2017

Conversation

@c-w
Copy link
Contributor

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

As per conversation around CatalystCode/project-fortis-pipeline#111

@c-w c-w requested a review from erikschlegel Aug 16, 2017
Copy link
Contributor

@jcjimenez jcjimenez left a comment

LGTM

Copy link
Contributor

@erikschlegel erikschlegel left a comment

LGTM. Please remember to update the cassandra ddls and the eventid datatype should remain as text

@c-w
Copy link
Contributor Author

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

@erikschlegel Yes, already posted the PR for the DDL update CatalystCode/project-fortis-pipeline#111

@c-w c-w merged commit 35d8a48 into master Aug 16, 2017
2 checks passed
@c-w c-w deleted the eventid branch Aug 16, 2017
@c-w c-w removed the in progress label Aug 16, 2017
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

4 participants