Uploaded image for project: 'HBase'
  1. HBase
  2. HBASE-14996 Some more API cleanup for 2.0
  3. HBASE-15583

Any HTableDescriptor we give out should be immutable

    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Resolved
    • Minor
    • Resolution: Fixed
    • 2.0.0
    • 2.0.0
    • None
    • None
    • Hide
      # The HTD got from Admin, AsyncAdmin, and Table is immutable.
      # DEFERRED_LOG_FLUSH is removed.
      # cleanup the deprecated construction of HTD
      Show
      # The HTD got from Admin, AsyncAdmin, and Table is immutable. # DEFERRED_LOG_FLUSH is removed. # cleanup the deprecated construction of HTD

    Description

      From enis in https://issues.apache.org/jira/browse/HBASE-15505:

      PS Should UnmodifyableHTableDescriptor be renamed to UnmodifiableHTableDescriptor?

      It should be named ImmutableHTableDescriptor to be consistent with collections naming. Let's do this as a subtask of the parent jira, not here. Thinking about it though, why would we return an Immutable HTD in HTable.getTableDescriptor() versus a mutable HTD in Admin.getTableDescriptor(). It does not make sense. Should we just get rid of the Immutable ones?
      We also have UnmodifyableHRegionInfo which is not used at the moment it seems.

      Attachments

        1. HBASE-15583.v0.patch
          80 kB
          Chia-Ping Tsai
        2. HBASE-15583.v1.patch
          80 kB
          Chia-Ping Tsai
        3. HBASE-15583.v2.patch
          198 kB
          Chia-Ping Tsai
        4. HBASE-15583.v3.patch
          213 kB
          Chia-Ping Tsai
        5. HBASE-15583.v4.patch
          213 kB
          Chia-Ping Tsai
        6. HBASE-15583.v5.patch
          213 kB
          Chia-Ping Tsai
        7. HBASE-15583.v6.patch
          213 kB
          Chia-Ping Tsai
        8. HBASE-15583.v7.patch
          213 kB
          Chia-Ping Tsai

        Issue Links

          Activity

            People

              chia7712 Chia-Ping Tsai
              gliptak Gabor Liptak
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved: