cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksey Yeschenko (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-12982) Customizable hint ttl has been removed
Date Fri, 23 Dec 2016 01:35:58 GMT


Aleksey Yeschenko commented on CASSANDRA-12982:


Re-encoding the vint here isn't really an issue, so no real need to avoid it.

I'm not a big fan, conceptually, of a conditional method with a name like {{deserializeIfLive}}
taking a {{force}} argument that basically turns it into an unconditional deserializer (and
making two or the arguments useless).

I much rather prefer some minor duplication to increased complexity but DRY-ness.

If you don't mind strongly, I'd like to give it another go once back from vacation/bank holidays
- to make it a bit simpler and conceptually right, also ideally in a way that would avoid
having to pass {{size}} to {{deserializeIfLive}}.

> Customizable hint ttl has been removed
> --------------------------------------
>                 Key: CASSANDRA-12982
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Blake Eggleston
>            Assignee: Blake Eggleston
>            Priority: Minor
>             Fix For: 3.x
> The cassandra.maxHintTTL property added in CASSANDRA-5988 was removed in the hint system
rewrite for 3.0.

This message was sent by Atlassian JIRA

View raw message