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

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    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 Soztutar 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

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

          People

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

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment