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

Move block cache parameters and references into single CacheConf class

VotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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

          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment