Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-24207

Wrong input argument order in TestCompactorMemLeak#assertMajorCompactionOK

    XMLWordPrintableJSON

    Details

    • Flags:
      Patch

      Description

      In TestCompactorMemLeak#assertMajorCompactionOK, the input argument order for 

      Assert.assertEquals(long expected, long actual) is wrong. This produced confusion when the test failed.

       

      Below is the original code:

      Assert.assertEquals(regions.size(), 1);
      HRegion region = regions.get(0);
      Assert.assertEquals(region.getStores().size(), 1);
      HStore store = region.getStore(FAMILY);
      Assert.assertEquals(store.getStorefilesCount(), 1);
      

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              ctest.team Ctest
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated: