cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Blake Eggleston (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-13217) minor bugs related to CASSANDRA-9143
Date Mon, 13 Feb 2017 23:38:41 GMT

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

Blake Eggleston commented on CASSANDRA-13217:
---------------------------------------------

bq. Looks like e1dda71 ninja fixed the log ordering, so your flipping is wrong in this patch
(because Dave already flipped it for you).

Removed the log order unfixing commit

> minor bugs related to CASSANDRA-9143
> ------------------------------------
>
>                 Key: CASSANDRA-13217
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-13217
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Blake Eggleston
>            Assignee: Blake Eggleston
>            Priority: Minor
>
> We found a few minor bugs found in an internal review:
> * -incorrect log argument order [here|https://github.com/apache/cassandra/blob/edcbef3e343778b4d5affe019f64c89da2a13aa2/src/java/org/apache/cassandra/streaming/compress/CompressedStreamReader.java#L75]-
> * {{SSTableReader#intersects}} should use Bounds, not Range ([here|https://github.com/apache/cassandra/blob/edcbef3e343778b4d5affe019f64c89da2a13aa2/src/java/org/apache/cassandra/io/sstable/format/SSTableReader.java#L1761])
> * {{CompactionStrategyManager#validateForCompaction}} doesn't prevent sstables from different
repair session from being compacted together [here|https://github.com/apache/cassandra/blob/edcbef3e343778b4d5affe019f64c89da2a13aa2/src/java/org/apache/cassandra/io/sstable/format/SSTableReader.java#L1761]



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Mime
View raw message