cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Brandon Williams (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-7712) temporary files need to be cleaned by unit tests
Date Sat, 13 Sep 2014 23:43:34 GMT


Brandon Williams commented on CASSANDRA-7712:

We can't change FU.deleteWithConfirm like this without impacting other non-test parts of the
code where it _should_ throw.  But deleteRecursive is pretty simple and could have a version
used inside a test util class, though it looks like this is localized to SchemaLoader.  Also
your IDE did some import re-writing that violate the code style.

> temporary files need to be cleaned by unit tests
> ------------------------------------------------
>                 Key: CASSANDRA-7712
>                 URL:
>             Project: Cassandra
>          Issue Type: Test
>          Components: Tests
>            Reporter: Michael Shuler
>            Priority: Minor
>              Labels: lhf
>             Fix For: 2.0.11
>         Attachments: CASSANDRA-7712_apache_cassandra_2.0.txt
> There are many unit test temporary files left behind after test runs. In the case of
CI servers, I have seen >70,000 files accumulate in /tmp over a period of time. Each unit
test should make an effort to remove its temporary files when the test is completed.
> My current unit test cleanup block:
> {noformat}
> # clean up after unit tests..
> rm -rf  /tmp/140*-0 /tmp/CFWith* /tmp/Counter1* /tmp/DescriptorTest* /tmp/Keyspace1*
>         /tmp/KeyStreamingTransferTestSpace* /tmp/SSTableExportTest* /tmp/SSTableImportTest*
>         /tmp/Standard1* /tmp/Statistics.db* /tmp/StreamingTransferTest* /tmp/ValuesWithQuotes*
>         /tmp/cassandra* /tmp/jna-* /tmp/ks-cf-ib-1-* /tmp/lengthtest* /tmp/liblz4-java*.so
/tmp/readtest* \
>         /tmp/set_length_during_read_mode* /tmp/set_negative_length* /tmp/snappy-*.so
> {noformat}

This message was sent by Atlassian JIRA

View raw message