[ 
https://issues.apache.org/jira/browse/SAMZA-407?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14123834#comment-14123834
 ] 

Chris Riccomini commented on SAMZA-407:
---------------------------------------

If we add this feature, we should definitely make it configurable, and off by 
default. Catching exceptions thrown in process() is dangerous, because it means 
we can drop output.

In my mind, this is a very similar feature to SAMZA-59. There are cases where 
developers would rather drop messages than fail outright when an exception is 
thrown in the process() method. To accommodate this, we can add a config to 
allow this, and track the failures in a metrics counter, based on exception 
type.

> Add metrics for counting exceptions by type
> -------------------------------------------
>
>                 Key: SAMZA-407
>                 URL: https://issues.apache.org/jira/browse/SAMZA-407
>             Project: Samza
>          Issue Type: Improvement
>          Components: container
>    Affects Versions: 0.8.0
>            Reporter: Chinmay Soman
>             Fix For: 0.8.0
>
>
> Currently, the container does not expose any metrics for tracking the number 
> of exceptions happening in the process() method. 
> We can catch all these exceptions and maintain a count per exception type 
> (before rethrowing the exception). These metrics can then be exposed for 
> further monitoring



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to