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

Uses latest fcrepo-snapshot build (3.0.0-RC1-SNAPSHOT) of metrics, which... #69

Merged
merged 1 commit into from May 30, 2013

Conversation

eddies
Copy link

@eddies eddies commented May 30, 2013

... addresses

https://github.com/codahale/metrics/issues/359
and, in turn,
https://github.com/codahale/metrics/issues/334

Removes MetricsService from fcrepo-kernel in favor of RegistryService from
fcrepo-metrics (not sure why we had this duplication to begin with)

…ich addresses

	https://github.com/codahale/metrics/issues/359
and, in turn,
	https://github.com/codahale/metrics/issues/334

Removes MetricsService from fcrepo-kernel in favor of RegistryService from
fcrepo-metrics (not sure why we had this duplication to begin with)
@ghost ghost assigned cbeer May 30, 2013
@eddies
Copy link
Author

eddies commented May 30, 2013

@cbeer I'm assigning you to this because I don't know if you had a reason for keeping fcrepo-kernel's MetricsService around given the functionally duplicative RegistryService frrom fcrepo-metrics. If not, please go ahead and merge.

cbeer added a commit that referenced this pull request May 30, 2013
Uses latest fcrepo-snapshot build (3.0.0-RC1-SNAPSHOT) of metrics, which...
@cbeer cbeer merged commit dfd4c12 into master May 30, 2013
@eddies eddies deleted the metrics-revisited branch May 30, 2013 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants