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

Dedupe locations/keywords/etc before aggregation #134

Merged
merged 2 commits into from Sep 11, 2017

Conversation

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

commented Sep 11, 2017

In the dashboard we want to show the number of events that match a particular location or keyword, not the number of times that the term was matched in the event.

@c-w c-w requested review from jcjimenez and erikschlegel Sep 11, 2017

@c-w c-w added the in progress label Sep 11, 2017

@jcjimenez
Copy link
Contributor

left a comment

LGTM

c-w added some commits Sep 11, 2017

Dedupe locations/keywords/etc before aggregation
In the dashboard we want to show the number of events that match a
particular location or keyword, not the number of times that the term
was matched in the event.

@c-w c-w force-pushed the terms-only-once branch from 65de267 to a2aeb7b Sep 11, 2017

@c-w c-w merged commit e015063 into master Sep 11, 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 terms-only-once branch Sep 11, 2017

@c-w c-w removed the in progress label Sep 11, 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.