Mailing-List: contact notifications-help@accumulo.apache.org; run by ezmlm
Precedence: bulk
Reply-To: jira@apache.org
Date: Sun, 1 Sep 2013 15:32:51 +0000 (UTC)
From: "Ted Yu (JIRA)" <jira@apache.org>
To: notifications@accumulo.apache.org
Message-ID: <JIRA.12659450.1374617534910.66213.1378049571572@arcas>
In-Reply-To: <JIRA.12659450.1374617534910@arcas>
References: <JIRA.12659450.1374617534910@arcas>
Subject: [jira] [Commented] (ACCUMULO-1604) ColumnAgeOffFilter doesn't
 validate options correctly
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit


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

Ted Yu commented on ACCUMULO-1604:
----------------------------------

>From trunk:
{code}
  public boolean validateOptions(Map<String,String> options) {
    if (super.validateOptions(options) == false)
      return false;
    try {
      this.ttls = new TTLSet(options);
    } catch (Exception e) {
      throw new IllegalArgumentException("bad TTL options", e);
    }
    return true;
{code}
Options for parent would be validated by super class.

Can you highlight the bug ?

Thanks
                
> ColumnAgeOffFilter doesn't validate options correctly
> -----------------------------------------------------
>
>                 Key: ACCUMULO-1604
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-1604
>             Project: Accumulo
>          Issue Type: Bug
>            Reporter: Christopher Tubbs
>              Labels: newbie
>             Fix For: 1.6.0
>
>
> The o.a.a.core.iterators.user.ColumnAgeOffFilter doesn't validate options correctly. It incorrectly assumes all options are for itself, and ignores the fact that some options may be for the parent class ("negate" option on Filter).

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira