New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add connection count field to beacons #297

Merged
merged 2 commits into from Dec 8, 2018

Conversation

Projects
None yet
2 participants
@lisaSW
Copy link
Collaborator

lisaSW commented Dec 8, 2018

#295
note: rebased on #293 branch

@lisaSW lisaSW requested review from ethack and meljbruno Dec 8, 2018

@lisaSW lisaSW changed the title 295 add conncount to beacons Add connection count field to beacons Dec 8, 2018

@ethack ethack changed the base branch from 269-update-uconns-collection to 293-update-hosts-collection Dec 8, 2018

@ethack

ethack approved these changes Dec 8, 2018

Copy link
Collaborator

ethack left a comment

The connection count is used in beacons and storing the value here trades space for time by avoiding a join with uconns when viewing the results.

@ethack ethack force-pushed the 295-add-conncount-to-beacons branch from 0cb55a8 to 5b3080d Dec 8, 2018

@ethack ethack merged commit f923861 into 293-update-hosts-collection Dec 8, 2018

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build is in progress
Details
continuous-integration/travis-ci/push The Travis CI build is in progress
Details

@ethack ethack deleted the 295-add-conncount-to-beacons branch Dec 8, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment