hbase-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "jiraposter@reviews.apache.org (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HBASE-4176) Exposing HBase Filters to the Thrift API
Date Thu, 18 Aug 2011 05:37:36 GMT

    [ https://issues.apache.org/jira/browse/HBASE-4176?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13086821#comment-13086821
] 

jiraposter@reviews.apache.org commented on HBASE-4176:
------------------------------------------------------


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/1326/
-----------------------------------------------------------

(Updated 2011-08-18 05:35:32.078889)


Review request for hbase, Todd Lipcon, Ted Yu, Michael Stack, and Jonathan Gray.


Changes
-------

Included the Filter Language document in the docbook. Please do let me know if I should add
more documentation or any formatting issues I can try and address.

P.S. Is trunk broken right now? I get an error saying - BaseMasterObserver is not abstract
and does not override abstract method stop(org.apache.hadoop.hbase.coprocessor.CoprocessorEnvironment)
in org.apache.hadoop.hbase.coprocessor.Coprocessor

This is completely unrelated to my diff - and I checked that the only difference b/w this
diff and the previous one that I had submitted is the change to book.xml. Thus I am going
ahead and uploading the patch.


Summary
-------

https://issues.apache.org/jira/browse/HBASE-4176: Exposing HBase Filters to the Thrift API

Currently, to use any of the filters, one has to explicitly add a scanner for the filter in
the Thrift API making it messy and long. 
With this patch, I am trying to add support for all the filters in a clean way. 
The user specifies a filter via a string. The string is parsed on the server to construct
the filter. More information can be found in the attached document named Filter Language

This patch is trying to extend and further the progress made by the patches in HBASE-1744

There is document attached to the HBASE-4176 JIRA that describes this patch in further detail


This addresses bug HBASE-4176.
    https://issues.apache.org/jira/browse/HBASE-4176


Diffs (updated)
-----

  /src/docbkx/book.xml 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ColumnCountGetFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ColumnPaginationFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ColumnPrefixFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ColumnRangeFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/CompareFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/DependentColumnFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/FamilyFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/FilterBase.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/FirstKeyOnlyFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/InclusiveStopFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/KeyOnlyFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/MultipleColumnPrefixFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/PageFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ParseConstants.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ParseFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/PrefixFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/QualifierFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/RowFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/SingleColumnValueExcludeFilter.java 1158977

  /src/main/java/org/apache/hadoop/hbase/filter/SingleColumnValueFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/TimestampsFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/filter/ValueFilter.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/thrift/ThriftServer.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/thrift/generated/Hbase.java 1158977 
  /src/main/java/org/apache/hadoop/hbase/thrift/generated/TScan.java 1158977 
  /src/main/resources/org/apache/hadoop/hbase/thrift/Hbase.thrift 1158977 
  /src/main/ruby/hbase/table.rb 1158977 
  /src/main/ruby/shell/commands/scan.rb 1158977 
  /src/test/java/org/apache/hadoop/hbase/filter/TestParseFilter.java 1158977 

Diff: https://reviews.apache.org/r/1326/diff


Testing
-------

patch includes one test: TestParseFilter.java


Thanks,

Anirudh



> Exposing HBase Filters to the Thrift API
> ----------------------------------------
>
>                 Key: HBASE-4176
>                 URL: https://issues.apache.org/jira/browse/HBASE-4176
>             Project: HBase
>          Issue Type: Improvement
>          Components: thrift
>            Reporter: Anirudh Todi
>            Assignee: Anirudh Todi
>            Priority: Minor
>         Attachments: Filter Language (3).xml, Filter Language(2).docx, Filter Language(2).xml,
Filter Language(3).docx, Filter Language.docx, HBASE-4176.patch, book.xml, book2.html, book2.xml
>
>
> Currently, to use any of the filters, one has to explicitly add a scanner for the filter
in the Thrift API making it messy and long. With this patch, I am trying to add support for
all the filters in a clean way. The user specifies a filter via a string. The string is parsed
on the server to construct the filter. More information can be found in the attached document
named Filter Language
> This patch is trying to extend and further the progress made by the patches in the HBASE-1744
JIRA (https://issues.apache.org/jira/browse/HBASE-1744)

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Mime
View raw message