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.

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

consider consolidating Poller, CorrelatedPoller, TranslatedPoller and UncorrelatedPoller #15

Open
jdugan1024 opened this issue Apr 1, 2014 · 3 comments

Comments

@jdugan1024
Copy link
Contributor

From jdu...@es.net on December 04, 2013 17:27:54

instead of assuming there is a correlator or translator poller_arg check and see and activate it if needed. I believe we can get to a single poller this way.

For example look at how I added a translator to the UncorrelatorPoller class in this commit:

changeset: 943:22e2de765699
parent: 941:2098af291b09
user: Jon M. Dugan jdugan@x1024.net
date: Wed Dec 04 17:14:56 2013 -0600
summary: add IfRefTranslator to clean up ifPhysAddress; remove ifPhysAddress cleanup from persister

Original issue: http://code.google.com/p/esmond/issues/detail?id=14

@jdugan1024
Copy link
Contributor Author

From jdu...@es.net on December 04, 2013 15:30:52

see r856ab79334fd3b5e84407224b139b8c5b18a6c67 for details

@jdugan1024
Copy link
Contributor Author

From jdu...@es.net on December 11, 2013 10:48:11

Labels: Milestone-Release1.1

@jdugan1024
Copy link
Contributor Author

From jdu...@es.net on December 11, 2013 10:50:45

Labels: -Type-Defect Type-Enhancement

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

No branches or pull requests

1 participant