Flume
  1. Flume
  2. FLUME-2037

testDataFilesRequiredByBackupNotDeleted is flakey

    Details

    • Type: Bug Bug
    • Status: Patch Available
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    1. FLUME-2037.patch
      4 kB
      Hari Shreedharan

      Issue Links

        Activity

        Hide
        Mike Percy added a comment -

        Hey Hari I am still getting failures on this test.

        Applied the patch, then

        $ cd flume-ng-channels/flume-file-channel
        $ for i in $(seq 1 100); do echo $i; mvn test -Dtest="TestFileChannelRestart#testDataFilesRequiredByBackupNotDeleted" || break; done
        

        I just found out that surefire supports that syntax... pretty sweet... anyway I am getting the following error message:

        $ head -10 ./target/surefire-reports/org.apache.flume.channel.file.TestFileChannelRestart.txt
        -------------------------------------------------------------------------------
        Test set: org.apache.flume.channel.file.TestFileChannelRestart
        -------------------------------------------------------------------------------
        Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.089 sec <<< FAILURE!
        testDataFilesRequiredByBackupNotDeleted(org.apache.flume.channel.file.TestFileChannelRestart)  Time elapsed: 0.946 sec  <<< FAILURE!
        java.lang.AssertionError
        	at org.junit.Assert.fail(Assert.java:92)
        	at org.junit.Assert.assertTrue(Assert.java:43)
        	at org.junit.Assert.assertTrue(Assert.java:54)
        	at org.apache.flume.channel.file.TestFileChannelRestart.testDataFilesRequiredByBackupNotDeleted(TestFileChannelRestart.java:795)
        

        I didn't investigate much, but I hacked up the patch a little and got this error message out of it after it failed on the 29th run:

        testDataFilesRequiredByBackupNotDeleted(org.apache.flume.channel.file.TestFileChannelRestart)  Time elapsed: 1.006 sec  <<< FAILURE!
        java.lang.AssertionError: origFiles.size() = 32, olderThanCheckpoint = 23
        	at org.junit.Assert.fail(Assert.java:93)
        	at org.junit.Assert.assertTrue(Assert.java:43)
        	at org.apache.flume.channel.file.TestFileChannelRestart.testDataFilesRequiredByBackupNotDeleted(TestFileChannelRestart.java:795)
        
        Show
        Mike Percy added a comment - Hey Hari I am still getting failures on this test. Applied the patch, then $ cd flume-ng-channels/flume-file-channel $ for i in $(seq 1 100); do echo $i; mvn test -Dtest="TestFileChannelRestart#testDataFilesRequiredByBackupNotDeleted" || break; done I just found out that surefire supports that syntax... pretty sweet... anyway I am getting the following error message: $ head -10 ./target/surefire-reports/org.apache.flume.channel.file.TestFileChannelRestart.txt ------------------------------------------------------------------------------- Test set: org.apache.flume.channel.file.TestFileChannelRestart ------------------------------------------------------------------------------- Tests run: 1, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.089 sec <<< FAILURE! testDataFilesRequiredByBackupNotDeleted(org.apache.flume.channel.file.TestFileChannelRestart) Time elapsed: 0.946 sec <<< FAILURE! java.lang.AssertionError at org.junit.Assert.fail(Assert.java:92) at org.junit.Assert.assertTrue(Assert.java:43) at org.junit.Assert.assertTrue(Assert.java:54) at org.apache.flume.channel.file.TestFileChannelRestart.testDataFilesRequiredByBackupNotDeleted(TestFileChannelRestart.java:795) I didn't investigate much, but I hacked up the patch a little and got this error message out of it after it failed on the 29th run: testDataFilesRequiredByBackupNotDeleted(org.apache.flume.channel.file.TestFileChannelRestart) Time elapsed: 1.006 sec <<< FAILURE! java.lang.AssertionError: origFiles.size() = 32, olderThanCheckpoint = 23 at org.junit.Assert.fail(Assert.java:93) at org.junit.Assert.assertTrue(Assert.java:43) at org.apache.flume.channel.file.TestFileChannelRestart.testDataFilesRequiredByBackupNotDeleted(TestFileChannelRestart.java:795)
        Hide
        Mike Percy added a comment -

        This one is still causing build failures.

        Show
        Mike Percy added a comment - This one is still causing build failures.
        Hide
        Mike Percy added a comment -

        Descoping from 1.4.0, not a blocker

        Show
        Mike Percy added a comment - Descoping from 1.4.0, not a blocker

          People

          • Assignee:
            Hari Shreedharan
            Reporter:
            Hari Shreedharan
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:

              Development