[
https://issues.apache.org/jira/browse/CASSANDRA-13698?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Paulo Motta updated CASSANDRA-13698:
------------------------------------
Resolution: Fixed
Fix Version/s: 3.11.3
3.0.17
4.0
Status: Resolved (was: Patch Available)
Thanks for the ping and sorry for the delay.
{quote}I got rid of testEchoedRow and testCompactionLog - it has been a long
time since we had EchoedRow in the codebase and we no longer have compaction
history stored in system tables.
{quote}
+1
{quote}I changed some of the superCFMDs to standardCFMDs
{quote}
+1
{quote}Not sure if you are able to explain why that is so, but otherwise I
think the test (as I have it) is fair.
{quote}
Actually this is because we [filter purgeable
tombstones|https://github.com/apache/cassandra/blob/39bcdcd32ea5dcfd372d4b3eaf10c1c3de0547fb/src/java/org/apache/cassandra/db/ReadCommand.java#L576]
during read so the tombstone was not visible on the test, but we want to keep
that check to make sure compaction will not get rid of the purgeable tombstone
when compacting only a subset of the sstables containing a row - it's fine to
do that on reads though because all SStables were queried. I ninja-ed a fix to
this
[here|https://github.com/pauloricardomg/cassandra/commits/fff93c5e97a8ca7f0035a672bc7bdcfa9c7a248d],
let me know if this looks good to you.
The rest of the patch LGTM, committed as
{{b8fb29a6ba1304010ac78df3cecba6ab2fece4cd}} to cassandra-3.0 and merged up to
master. Great job, thanks!
> Reinstate or get rid of unit tests with multiple compaction strategies
> ----------------------------------------------------------------------
>
> Key: CASSANDRA-13698
> URL: https://issues.apache.org/jira/browse/CASSANDRA-13698
> Project: Cassandra
> Issue Type: Test
> Components: Testing
> Reporter: Paulo Motta
> Assignee: Lerh Chuan Low
> Priority: Minor
> Labels: lhf
> Fix For: 4.0, 3.0.17, 3.11.3
>
> Attachments: 13698-3.0.txt, 13698-3.11.txt, 13698-trunk.txt
>
>
> At some point there were (anti-)compaction tests with multiple compaction
> strategy classes, but now it's only tested with {{STCS}}:
> *
> [AnticompactionTest|https://github.com/apache/cassandra/blob/8b3a60b9a7dbefeecc06bace617279612ec7092d/test/unit/org/apache/cassandra/db/compaction/AntiCompactionTest.java#L247]
> *
> [CompactionsTest|https://github.com/apache/cassandra/blob/8b3a60b9a7dbefeecc06bace617279612ec7092d/test/unit/org/apache/cassandra/db/compaction/CompactionsTest.java#L85]
> We should either reinstate these tests or decide they are not important and
> remove the unused parameter.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
