You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@samza.apache.org by "Chris Riccomini (JIRA)" <ji...@apache.org> on 2014/09/06 01:46:30 UTC

[jira] [Commented] (SAMZA-407) Add metrics for counting exceptions by type

    [ 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)