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

Chris Riccomini commented on SAMZA-540:
---------------------------------------

I might be a bit confused, but isn't this ticket to add latency metrics in 
OffsetManager? Did you mean timer {} or just offset tracking?

{noformat}
info("Checkpointed offset %s for %s." format (offset, systemStreamPartition))
{noformat}

This comment is a bit confusing. It sounds like it's saying that the offset 
manager sent a checkpoint offset message, but it's actually trying to say that 
the offset that it read, and is going to use, is offset X.  Maybe "Checkpointed 
offset is currently %s for %s."

> Expose latency related metrics in OffsetManager
> -----------------------------------------------
>
>                 Key: SAMZA-540
>                 URL: https://issues.apache.org/jira/browse/SAMZA-540
>             Project: Samza
>          Issue Type: Improvement
>          Components: metrics
>    Affects Versions: 0.8.0
>            Reporter: Yan Fang
>            Assignee: Renato Javier MarroquĂ­n Mogrovejo
>              Labels: newbie
>             Fix For: 0.10.0
>
>         Attachments: SAMZA-540.1.patch, SAMZA-540.4.patch, SAMZA-540.5.patch
>
>
> Follow-up to SAMZA-503
> Expose checkpointed offset, max offset, and max offset - checkpointed offset 
> metrics in OffsetManager.



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

Reply via email to