cassandra-commits mailing list archives

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

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

Aleksey Yeschenko commented on CASSANDRA-12982:
-----------------------------------------------

Thanks.

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: https://issues.apache.org/jira/browse/CASSANDRA-12982
>             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
(v6.3.4#6332)

Mime
View raw message