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

Move block cache parameters and references into single CacheConf class

    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • None
    • 0.92.0
    • io
    • None
    • Reviewed
    • Relocates all cache configuration options and constants into a new CacheConfig class. Modifies lots of constructors from Store/StoreFile down to HFile.

    Description

      From StoreFile down to HFile, we currently use a boolean argument for each of the various block cache configuration parameters that exist. The number of parameters is going to continue to increase as we look at compressed cache, delta encoding, and more specific L1/L2 configuration. Every new config currently requires changing many constructors because it introduces a new boolean.

      We should move everything into a single class so that modifications are much less disruptive.

      Attachments

        1. ASF.LICENSE.NOT.GRANTED--D1341.1.patch
          149 kB
          Phabricator
        2. CacheConfig92-v8.patch
          124 kB
          Jonathan Gray
        3. HBASE-4422-FINAL-branch92.patch
          129 kB
          Jonathan Gray
        4. HBASE-4422-FINAL-trunk.patch
          129 kB
          Jonathan Gray

        Issue Links

          Activity

            People

              streamy Jonathan Gray
              streamy Jonathan Gray
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: