HBase
  1. HBase
  2. HBASE-370

[Hbase Shell] Remove the algebra package

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Trivial Trivial
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      After chatting with stack, we decided to remove the algebra package becuase it is outgrown from hbase.

      1. delete_v2.patch
        365 kB
        Edward J. Yoon
      2. delete.patch
        170 kB
        Edward J. Yoon

        Activity

        Hide
        Edward J. Yoon added a comment -

        Also, I'd like to some refactor in this patch.

        Show
        Edward J. Yoon added a comment - Also, I'd like to some refactor in this patch.
        Hide
        Edward J. Yoon added a comment -

        I'll upload the patch tomorrow.

        Show
        Edward J. Yoon added a comment - I'll upload the patch tomorrow.
        Hide
        Edward J. Yoon added a comment -

        submitting

        Show
        Edward J. Yoon added a comment - submitting
        Hide
        Edward J. Yoon added a comment -

        If we use the Map(select-from-where) and Reduce (sort, count, sum, ... , etc) in hbase shell, hql will be more SQL-like.
        Please review one more time before committing, stack.

        Thanks.

        Show
        Edward J. Yoon added a comment - If we use the Map(select-from-where) and Reduce (sort, count, sum, ... , etc) in hbase shell, hql will be more SQL-like. Please review one more time before committing, stack. Thanks.
        Hide
        stack added a comment -

        Patch looks good to me. I tried it and the help looks coherent and things still basically work. +1.

        Show
        stack added a comment - Patch looks good to me. I tried it and the help looks coherent and things still basically work. +1.
        Hide
        Hadoop QA added a comment -

        +1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12372373/delete_v2.patch
        against trunk revision r608234.

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

        javadoc +1. The javadoc tool did not generate any warning messages.

        javac +1. The applied patch does not generate any new compiler warnings.

        findbugs +1. The patch does not introduce any new Findbugs warnings.

        core tests +1. The patch passed core unit tests.

        contrib tests +1. The patch passed contrib unit tests.

        Test results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/testReport/
        Findbugs warnings: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
        Checkstyle results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/artifact/trunk/build/test/checkstyle-errors.html
        Console output: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/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/12372373/delete_v2.patch against trunk revision r608234. @author +1. The patch does not contain any @author tags. javadoc +1. The javadoc tool did not generate any warning messages. javac +1. The applied patch does not generate any new compiler warnings. findbugs +1. The patch does not introduce any new Findbugs warnings. core tests +1. The patch passed core unit tests. contrib tests +1. The patch passed contrib unit tests. Test results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/testReport/ Findbugs warnings: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/artifact/trunk/build/test/checkstyle-errors.html Console output: http://lucene.zones.apache.org:8080/hudson/job/Hadoop-Patch/1446/console This message is automatically generated.
        Hide
        stack added a comment -

        Committed. Thanks for the patch Edward.

        Show
        stack added a comment - Committed. Thanks for the patch Edward.
        Hide
        stack added a comment -

        This removes functionality added by HADOOP-1608

        Show
        stack added a comment - This removes functionality added by HADOOP-1608

          People

          • Assignee:
            Unassigned
            Reporter:
            Edward J. Yoon
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development