You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@accumulo.apache.org by "David Medinets (Created) (JIRA)" <ji...@apache.org> on 2012/03/29 04:31:24 UTC

[jira] [Created] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Document what replaces the deprecated PerColumnIteratorConfig class.
--------------------------------------------------------------------

                 Key: ACCUMULO-502
                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
             Project: Accumulo
          Issue Type: Improvement
            Reporter: David Medinets
            Priority: Minor


I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:

$ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
$ svn log -r 1178656:1178654
r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
ACCUMULO-1 initial checkin of source code

What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?




--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Re: [jira] [Created] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by Keith Turner <ke...@deenlo.com>.
I think the only use for PerColumnIteratorConfig is to configure
aggregators, which are also deprecated in favor of combiners.  So I
think PerColumnIteratorConfig was deprecated when aggregators were
deprecated.

Keith

On Wed, Mar 28, 2012 at 10:31 PM, David Medinets (Created) (JIRA)
<ji...@apache.org> wrote:
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: David Medinets
>            Priority: Minor
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
>
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
>
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?
>
>
>
>
> --
> This message is automatically generated by JIRA.
> If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
> For more information on JIRA, see: http://www.atlassian.com/software/jira
>
>

[jira] [Updated] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "John Vines (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Vines updated ACCUMULO-502:
--------------------------------

    Resolution: Fixed
        Status: Resolved  (was: Patch Available)
    
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "John Vines (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

John Vines updated ACCUMULO-502:
--------------------------------

    Assignee: Josh Elser  (was: David Medinets)

I don't remember if this discussion was made when you were around David, but we're going to keep tickets assigned to those who supply the patch (if applicable). This way, contributors who are not committers, like Josh, have some documentation of their contributions on our JIRA. This is important to us when making decisions to promote contributors to a committer status.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "Josh Elser (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser updated ACCUMULO-502:
--------------------------------

        Fix Version/s: 1.4.1
             Assignee: Josh Elser
    Affects Version/s: 1.4.1
               Status: Patch Available  (was: Open)

Attached.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "Josh Elser (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13268070#comment-13268070 ] 

Josh Elser commented on ACCUMULO-502:
-------------------------------------

David: did Keith's response answer your question, is the accompanying Javadoc addition sufficient, and can this ticket be closed?
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "Keith Turner (Commented) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13241203#comment-13241203 ] 

Keith Turner commented on ACCUMULO-502:
---------------------------------------

Responding in the correct place, I had responded on the mailing list overlooking that this was a ticket.

I think the only use for PerColumnIteratorConfig is to configure aggregators, which are also deprecated in favor of combiners.  So I think PerColumnIteratorConfig was deprecated when aggregators were deprecated.  

The replace would be using ColumnSet and Combiners.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: David Medinets
>            Priority: Minor
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Issue Comment Edited] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "Keith Turner (Issue Comment Edited) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13241203#comment-13241203 ] 

Keith Turner edited comment on ACCUMULO-502 at 3/29/12 1:16 PM:
----------------------------------------------------------------

Responding in the correct place, I had responded on the mailing list overlooking that this was a ticket.

I think the only use for PerColumnIteratorConfig is to configure aggregators, which are deprecated in favor of combiners.  So I think PerColumnIteratorConfig was deprecated when aggregators were deprecated.  

The replacement would be using ColumnSet and Combiners.
                
      was (Author: kturner):
    Responding in the correct place, I had responded on the mailing list overlooking that this was a ticket.

I think the only use for PerColumnIteratorConfig is to configure aggregators, which are also deprecated in favor of combiners.  So I think PerColumnIteratorConfig was deprecated when aggregators were deprecated.  

The replace would be using ColumnSet and Combiners.
                  
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>            Reporter: David Medinets
>            Priority: Minor
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Updated] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "Josh Elser (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Josh Elser updated ACCUMULO-502:
--------------------------------

    Attachment: ACCUMULO-502-PerColumnIteratorConfig-javadoc

Add javadoc to reference replacement Column class in IteratorSetting and the SetColumns method on the Combiner class.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Commented] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "David Medinets (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13273014#comment-13273014 ] 

David Medinets commented on ACCUMULO-502:
-----------------------------------------

Ok.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: Josh Elser
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

[jira] [Closed] (ACCUMULO-502) Document what replaces the deprecated PerColumnIteratorConfig class.

Posted by "David Medinets (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/ACCUMULO-502?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

David Medinets closed ACCUMULO-502.
-----------------------------------

    Assignee: David Medinets  (was: Josh Elser)

Thanks for updating the class.
                
> Document what replaces the deprecated PerColumnIteratorConfig class.
> --------------------------------------------------------------------
>
>                 Key: ACCUMULO-502
>                 URL: https://issues.apache.org/jira/browse/ACCUMULO-502
>             Project: Accumulo
>          Issue Type: Improvement
>    Affects Versions: 1.4.1
>            Reporter: David Medinets
>            Assignee: David Medinets
>            Priority: Minor
>             Fix For: 1.4.1
>
>         Attachments: ACCUMULO-502-PerColumnIteratorConfig-javadoc
>
>
> I pulled a new copy of trunk and noticed that a deprecated class is being used. I tried some code spelunking:
> $ svn -v blame core/src/main/java/org/apache/accumulo/core/iterators/conf/PerColumnIteratorConfig.java
> 1178655    kturner 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011)  * @deprecated since 1.4
> $ svn log -r 1178656:1178654
> r1178655 | kturner | 2011-10-03 20:46:07 -0400 (Mon, 03 Oct 2011) | 1 line
> ACCUMULO-1 initial checkin of source code
> What should be used in place of PerColumnIteratorConfig? Can that information be added to the PerColumnIteratorConfig.java file?

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira