cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sylvain Lebresne (JIRA)" <>
Subject [jira] [Updated] (CASSANDRA-4980) StorageServiceClientTest/RecoveryManager2Test fail on 1.2.0 and above branch
Date Wed, 21 Nov 2012 17:27:59 GMT


Sylvain Lebresne updated CASSANDRA-4980:

    Attachment: 0002-Clean-up-the-commit-log-before-RecoveryManager2Test.txt

I'd rather avoid that if possible.

The problem in StorageServiceClientTest is imo a bug that this commit just happens to reveal.
Namely, that for some reason {{SS.initClient()}} calls updateVersionAndAnnounce. But since
fat clients are supposed to not have local tables, they will in particular have no schema
and this is useless. If for some weird reason we really need fat client to send a 'I have
no schema' on gossip, let's call {{MigrationManager.passiveAnnounce(Schema.emptyVersion)}}
directly (but I don't see why we would need that).

As for RecoveryManager2Test, it's just that this test was somehow assuming the commit log
was empty when the test started. Which was the case because we almost always flush when we
write the system table, but it happens that for writing the schemaVersion in the local table
I figured flushing was overkill (since it's only for client sake and we don't need to flush
to have it visible).

Attaching simple fixes for both problems.
> StorageServiceClientTest/RecoveryManager2Test fail on 1.2.0 and above branch
> ----------------------------------------------------------------------------
>                 Key: CASSANDRA-4980
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>    Affects Versions: 1.2.0 beta 2
>            Reporter: Yuki Morishita
>         Attachments: 0001-fix-StorageServiceClientTest.txt, 0002-Clean-up-the-commit-log-before-RecoveryManager2Test.txt
> Looks like change in c4cca2d8bba20a7651b956e1893727391bf5f10a (store schema_version to
system.local) broke both StorageServiceClientTest and RecoveryManager2Test.
> StorageServiceClientTest assert data directories are not created in client mode but this
change actually creates data directories. RecoveryManager2Test fails with "junit.framework.AssertionFailedError:
Expecting only 1 replayed mutation, got 10" error and I think extra commit log also comes
from this insert to system.local.

This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see:

View raw message