Details

    Description

      commitlog_test.py:TestCommitLog.test_compression_error fails on Windows under CassCI. It fails in a number of different ways. Here, it looks like reading the CRC fails:

      http://cassci.datastax.com/view/win32/job/cassandra-3.0_dtest_win32/100/testReport/commitlog_test/TestCommitLog/test_compression_error/

      Here, I believe it fails when trying to validate the CRC header:

      http://cassci.datastax.com/view/win32/job/cassandra-3.0_dtest_win32/99/testReport/commitlog_test/TestCommitLog/test_compression_error/
      https://github.com/riptano/cassandra-dtest/blob/master/commitlog_test.py#L497

      Here's another failure where the header has a Q written in it instead of a closing brace:

      http://cassci.datastax.com/view/win32/job/cassandra-3.0_dtest_win32/91/testReport/junit/commitlog_test/TestCommitLog/test_compression_error/
      https://github.com/riptano/cassandra-dtest/blob/master/commitlog_test.py#L513

      bdeggleston Do I remember correctly that you wrote this test? Can you take this on?

      Attachments

        Activity

          mambocab I did write the test, but I don't think I'm going to have the time to look at this within the next week or so. At first glance though, this doesn't look like a problem with the test, but a problem with writing the commit log to disk on windows. Especially the Q replacing the closing brace on the last one.

          bdeggleston Blake Eggleston added a comment - mambocab I did write the test, but I don't think I'm going to have the time to look at this within the next week or so. At first glance though, this doesn't look like a problem with the test, but a problem with writing the commit log to disk on windows. Especially the Q replacing the closing brace on the last one.
          mambocab Jim Witschey added a comment -

          bdeggleston Thanks for having a look. JoshuaMcKenzie Do you know who else might be able to take this on?

          mambocab Jim Witschey added a comment - bdeggleston Thanks for having a look. JoshuaMcKenzie Do you know who else might be able to take this on?

          I'll dig into this tomorrow. At the very least we need to confirm whether or not this is a real problem on Windows or just a test problem before we cut the release.

          JoshuaMcKenzie Joshua McKenzie added a comment - I'll dig into this tomorrow. At the very least we need to confirm whether or not this is a real problem on Windows or just a test problem before we cut the release.

          Flaky and usually passes, so I'm not worried about this for 3.0.0 GA.

          JoshuaMcKenzie Joshua McKenzie added a comment - Flaky and usually passes , so I'm not worried about this for 3.0.0 GA.

          People

            Unassigned Unassigned
            mambocab Jim Witschey
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated: