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

Client Public API should not have PB objects in 2.0

VotersWatch issueWatchersLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Sub-task
    • Status: Closed
    • Blocker
    • Resolution: Fixed
    • None
    • 2.0.0
    • None
    • None
    • Reviewed

    Description

      Some more cleanup for the parent jira.
      We have leaked some PB structs in Admin (and possible other places).

      We should clean up these API before 2.0.

      Examples include:

        AdminProtos.GetRegionInfoResponse.CompactionState getCompactionState(final TableName tableName)
          throws IOException;
         
          ....
        void snapshot(final String snapshotName,
            final TableName tableName,
            HBaseProtos.SnapshotDescription.Type type) throws IOException, SnapshotCreationException,
            IllegalArgumentException;
         ....
        MasterProtos.SnapshotResponse takeSnapshotAsync(HBaseProtos.SnapshotDescription snapshot)
            throws IOException, SnapshotCreationException;
      

      Attachments

        1. HBASE-15174_1.patch
          25 kB
          ramkrishna.s.vasudevan
        2. HBASE-15174_2.patch
          28 kB
          ramkrishna.s.vasudevan
        3. HBASE-15174_3.patch
          11 kB
          ramkrishna.s.vasudevan
        4. HBASE-15174_4.patch
          11 kB
          ramkrishna.s.vasudevan
        5. HBASE-15174_5.patch
          11 kB
          ramkrishna.s.vasudevan
        6. HBASE-15174.patch
          6 kB
          ramkrishna.s.vasudevan

        Issue Links

        Activity

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

          People

            ram_krish ramkrishna.s.vasudevan
            enis Enis Soztutar
            Votes:
            0 Vote for this issue
            Watchers:
            13 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment