You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@hbase.apache.org by "stack (JIRA)" <ji...@apache.org> on 2011/07/12 16:56:59 UTC

[jira] [Assigned] (HBASE-4086) documentation/javadoc error in SingleColumnValueFilter constructor

     [ https://issues.apache.org/jira/browse/HBASE-4086?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

stack reassigned HBASE-4086:
----------------------------

    Assignee: Doug Meil

You want to have a go at this one Doug?  If not, assign back to me.  Good on you.

> documentation/javadoc error in SingleColumnValueFilter constructor
> ------------------------------------------------------------------
>
>                 Key: HBASE-4086
>                 URL: https://issues.apache.org/jira/browse/HBASE-4086
>             Project: HBase
>          Issue Type: Bug
>          Components: documentation
>    Affects Versions: 0.90.3
>         Environment: all
>            Reporter: nkeywal
>            Assignee: Doug Meil
>            Priority: Trivial
>
> The behaviour when the column is not found is documented differently in the constructor and in the setter. The constructor is actually wrong: by default, when the column is not found, the row is emitted (may be the opposite would be better, but it's another question)
> SingleColumnValueFilter
> public SingleColumnValueFilter(byte[] family,
>                                byte[] qualifier,
>                                CompareFilter.CompareOp compareOp,
>                                byte[] value)
> Constructor for binary compare of the value of a single column. If the column is found and the condition passes, all columns of the row will be emitted. If the column is not found or the condition fails, the row will not be emitted.
> setFilterIfMissing
> public void setFilterIfMissing(boolean filterIfMissing)
> Set whether entire row should be filtered if column is not found.
> If true, the entire row will be skipped if the column is not found.
> If false, the row will pass if the column is not found. This is default. 
> Possible correction for the constructor documentation:
> If the column is found and the condition passes, all columns of the row will be emitted. If the condition fails, the row will not be emitted. The behavior when the column is not found is defined by setFilterIfMissing.

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