HBase
  1. HBase
  2. HBASE-7870

Delete(byte [] row, long timestamp) constructor is deprecate and should not.

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 0.94.5
    • Fix Version/s: 0.94.6
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Most probably a cut&past issue. Patch is coming.

      1. HBASE-7870-v0-0.94.patch
        0.8 kB
        Jean-Marc Spaggiari

        Activity

        Hide
        Jean-Marc Spaggiari added a comment -

        Fixing:
        public Delete(byte [] row) which should not use lockID constructor
        public Delete(byte [] row, long timestamp) comments which are making it deprecated when it should not since not using LockID.

        Show
        Jean-Marc Spaggiari added a comment - Fixing: public Delete(byte [] row) which should not use lockID constructor public Delete(byte [] row, long timestamp) comments which are making it deprecated when it should not since not using LockID.
        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12569808/HBASE-7870-v0-0.94.patch
        against trunk revision .

        +1 @author. The patch does not contain any @author tags.

        -1 tests included. The patch doesn't appear to include any new or modified tests.
        Please justify why no new tests are needed for this patch.
        Also please list what manual steps were performed to verify this patch.

        -1 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/4453//console

        This message is automatically generated.

        Show
        Hadoop QA added a comment - -1 overall . Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12569808/HBASE-7870-v0-0.94.patch against trunk revision . +1 @author . The patch does not contain any @author tags. -1 tests included . The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. -1 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/4453//console This message is automatically generated.
        Hide
        Ted Yu added a comment -

        Good catch.
        +1

        Show
        Ted Yu added a comment - Good catch. +1
        Hide
        Nicolas Liochon added a comment -

        +1. I checked, it's not in trunk at all as well.

        Show
        Nicolas Liochon added a comment - +1. I checked, it's not in trunk at all as well.
        Hide
        Nicolas Liochon added a comment -

        committed to 0.94, thanks for the patch, Jean-Marc!

        Show
        Nicolas Liochon added a comment - committed to 0.94, thanks for the patch, Jean-Marc!
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94 #850 (See https://builds.apache.org/job/HBase-0.94/850/)
        HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176)

        Result = SUCCESS
        nkeywal :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Show
        Hudson added a comment - Integrated in HBase-0.94 #850 (See https://builds.apache.org/job/HBase-0.94/850/ ) HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176) Result = SUCCESS nkeywal : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94-security #111 (See https://builds.apache.org/job/HBase-0.94-security/111/)
        HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176)

        Result = SUCCESS
        nkeywal :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Show
        Hudson added a comment - Integrated in HBase-0.94-security #111 (See https://builds.apache.org/job/HBase-0.94-security/111/ ) HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176) Result = SUCCESS nkeywal : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94-security-on-Hadoop-23 #12 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/12/)
        HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176)

        Result = FAILURE
        nkeywal :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Show
        Hudson added a comment - Integrated in HBase-0.94-security-on-Hadoop-23 #12 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/12/ ) HBASE-7870 Delete(byte [] row, long timestamp) constructor is deprecate and should not. (Jean-Marc Spaggiari) (Revision 1448176) Result = FAILURE nkeywal : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/client/Delete.java
        Hide
        Jean-Marc Spaggiari added a comment -

        This has been commited in Feb... Can we mark it as resolved and add the release version?

        Show
        Jean-Marc Spaggiari added a comment - This has been commited in Feb... Can we mark it as resolved and add the release version?

          People

          • Assignee:
            Jean-Marc Spaggiari
            Reporter:
            Jean-Marc Spaggiari
          • Votes:
            0 Vote for this issue
            Watchers:
            5 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development