Details
-
Type:
Improvement
-
Status: Resolved
-
Priority:
Major
-
Resolution: Fixed
-
Affects Version/s: 1.4.0
-
Component/s: Sinks+Sources
-
Labels:None
Description
Otherwise the each thread will have it's own registry which isn't great for getting a sense of the overall metrics, i.e. aggregated across all threads.
Activity
- All
- Comments
- Work Log
- History
- Activity
- Transitions
add some more metrics
Show
wolfgang hoschek
added a comment - add some more metrics
+1. Looks good. Running tests and committing.
Show
Hari Shreedharan
added a comment - +1. Looks good. Running tests and committing.
Committed, rev: c420fad. Thanks Wolfgang!
Show
Hari Shreedharan
added a comment - Committed, rev: c420fad. Thanks Wolfgang!
SUCCESS: Integrated in flume-trunk #514 (See https://builds.apache.org/job/flume-trunk/514/)
FLUME-2213. MorphlineInterceptor should share metric registry across threads for better (aggregate) reporting (hshreedharan: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=c420fad5d03dc8d17dce7fe3e59bf3b742f3d22d)
- flume-ng-sinks/flume-ng-morphline-solr-sink/src/main/java/org/apache/flume/sink/solr/morphline/MorphlineHandlerImpl.java
Show
Hudson
added a comment - SUCCESS: Integrated in flume-trunk #514 (See https://builds.apache.org/job/flume-trunk/514/ )
FLUME-2213 . MorphlineInterceptor should share metric registry across threads for better (aggregate) reporting (hshreedharan: http://git-wip-us.apache.org/repos/asf/flume/repo?p=flume.git&a=commit&h=c420fad5d03dc8d17dce7fe3e59bf3b742f3d22d )
flume-ng-sinks/flume-ng-morphline-solr-sink/src/main/java/org/apache/flume/sink/solr/morphline/MorphlineHandlerImpl.java
better patch, now also shared registry among sinks that use the same morphlineFile and morphlineId