cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Joel Knighton (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-11168) Hint Metrics are updated even if hinted_hand-offs=false
Date Thu, 10 Mar 2016 19:11:40 GMT


Joel Knighton updated CASSANDRA-11168:
    Status: Open  (was: Patch Available)

The latest patch is pretty close - it just needs a few tweaks.

# It looks like the boolean condition around hintedHandoffEnabled() is inverted - if we're
moving all the following code inside the if, it should only be run when hinted handoff is
enabled, not the opposite.
# It looks like we still increment the metric when we decide not to hint because hints are
disabled for a specific datacenter. We shouldn't do this.

The version from your comment above doesn't have these problems; if you want me to make a
patch out of that, I can.

> Hint Metrics are updated even if hinted_hand-offs=false
> -------------------------------------------------------
>                 Key: CASSANDRA-11168
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Anubhav Kale
>            Assignee: Anubhav Kale
>            Priority: Minor
>         Attachments: 0001-Hinted-Handoff-Fix.patch, 0001-Hinted-Handoff-fix-2_2.patch,
> In our PROD logs, we noticed a lot of hint metrics even though we have disabled hinted
> The reason is StorageProxy.ShouldHint has an inverted if condition. 
> We should also wrap the if (hintWindowExpired) block in if (DatabaseDescriptor.hintedHandoffEnabled()).
> The fix is easy, and I can provide a patch.

This message was sent by Atlassian JIRA

View raw message