Uploaded image for project: 'Bookkeeper'
  1. Bookkeeper
  2. BOOKKEEPER-460

LedgerDeleteTest checks wrong place for log file

    XMLWordPrintableJSON

Details

    • Test
    • Status: Closed
    • Trivial
    • Resolution: Fixed
    • None
    • 4.2.0
    • bookkeeper-server
    • None

    Description

      A minor problem, it seems forgot to update LedgerDeleteTest after upgrade bookkeeper filesystem.

      // Verify that the first entry log (0.log) has been deleted from all of the Bookie Servers.
      for (File ledgerDirectory : tmpDirs) {
          for (File f : ledgerDirectory.listFiles()) {
              assertFalse("Found the entry log file (0.log) that should have been deleted in ledgerDirectory: " 
                  + ledgerDirectory, f.isFile() && f.getName().equals("0.log"));
          }
      }
      

      Solution:
      Just reuse the checkLogFiles method in CompactionTest.

      Attachments

        1. BOOKKEEPER-460.patch
          9 kB
          Fangmin Lv
        2. BOOKKEEPER-460.diff
          8 kB
          Fangmin Lv
        3. BOOKKEEPER-460.diff
          9 kB
          Fangmin Lv

        Activity

          People

            lvfangmin Fangmin Lv
            lvfangmin Fangmin Lv
            Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: