Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Send an email to all SavedSearches which cover the commit that’s been commented on. #7

Closed
philc opened this issue Sep 1, 2011 · 3 comments
Closed

Send an email to all SavedSearches which cover the commit that’s been commented on. #7

philc opened this issue Sep 1, 2011 · 3 comments
Assignees
@philc
Labels
feature request p2

Comments

@philc
Copy link
Contributor

philc commented Sep 1, 2011

I'll take this.
@ghost ghost assigned philc Sep 4, 2011
@philc
Copy link
Contributor Author

philc commented Sep 5, 2011

This has been completed in 6440fe1. It just needs some performance testing to ensure it's not prohibitively expensive.

@philc
Copy link
Contributor Author

philc commented Sep 6, 2011

This is now live at Ooyala. I'd like to run some numbers and record how long each of these saved-searches is taking to see if it has introduced egregious performance overhead.

@philc
Copy link
Contributor Author

philc commented Sep 14, 2011

Closing this; looks good.

@philc philc closed this as completed Sep 14, 2011
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@philc philc

Labels
feature request p2
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@philc