You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@mahout.apache.org by "tom pierce (Created) (JIRA)" <ji...@apache.org> on 2011/12/11 21:00:47 UTC

[jira] [Created] (MAHOUT-921) FPG uses a lot of boxed primitives - this patch eliminates a bunch of List

FPG uses a lot of boxed primitives - this patch eliminates a bunch of List<Integer> 
------------------------------------------------------------------------------------

                 Key: MAHOUT-921
                 URL: https://issues.apache.org/jira/browse/MAHOUT-921
             Project: Mahout
          Issue Type: Improvement
          Components: Frequent Itemset/Association Rule Mining
    Affects Versions: 0.6
            Reporter: tom pierce
            Priority: Minor


TransactionTree uses List<Integer> internally and as part of its API; this patch changes that and pushes the use of List<Integer> down to FPGrowth.java.  This can be converted if the perf bug can be fixed (MAHOUT-890).  I will also provide an alternate base FPGrowth implementation.

Please note that this patch assumes the patch attached to MAHOUT-920 has already been applied.

--
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] [Resolved] (MAHOUT-921) FPG uses a lot of boxed primitives - this patch eliminates a bunch of List

Posted by "Robin Anil (Resolved) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAHOUT-921?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robin Anil resolved MAHOUT-921.
-------------------------------

       Resolution: Fixed
    Fix Version/s: 0.6
    
> FPG uses a lot of boxed primitives - this patch eliminates a bunch of List<Integer> 
> ------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-921
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-921
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Frequent Itemset/Association Rule Mining
>    Affects Versions: 0.6
>            Reporter: tom pierce
>            Assignee: Robin Anil
>            Priority: Minor
>             Fix For: 0.6
>
>         Attachments: MAHOUT-921.patch
>
>
> TransactionTree uses List<Integer> internally and as part of its API; this patch changes that and pushes the use of List<Integer> down to FPGrowth.java.  This can be converted if the perf bug can be fixed (MAHOUT-890).  I will also provide an alternate base FPGrowth implementation.
> Please note that this patch assumes the patch attached to MAHOUT-920 has already been applied.

--
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] (MAHOUT-921) FPG uses a lot of boxed primitives - this patch eliminates a bunch of List

Posted by "tom pierce (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAHOUT-921?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

tom pierce updated MAHOUT-921:
------------------------------

    Attachment: MAHOUT-921.patch

Note patch assumes MAHOUT-920 has been applied!
                
> FPG uses a lot of boxed primitives - this patch eliminates a bunch of List<Integer> 
> ------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-921
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-921
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Frequent Itemset/Association Rule Mining
>    Affects Versions: 0.6
>            Reporter: tom pierce
>            Priority: Minor
>         Attachments: MAHOUT-921.patch
>
>
> TransactionTree uses List<Integer> internally and as part of its API; this patch changes that and pushes the use of List<Integer> down to FPGrowth.java.  This can be converted if the perf bug can be fixed (MAHOUT-890).  I will also provide an alternate base FPGrowth implementation.
> Please note that this patch assumes the patch attached to MAHOUT-920 has already been applied.

--
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] (MAHOUT-921) FPG uses a lot of boxed primitives - this patch eliminates a bunch of List

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

Hudson commented on MAHOUT-921:
-------------------------------

Integrated in Mahout-Quality #1307 (See [https://builds.apache.org/job/Mahout-Quality/1307/])
    MAHOUT-890 MAHOUT-920 MAHOUT-921 MAHOUT-927 FpGrowth2 Implementation and fixes to existing Mapreduce classes in pfpgrowth

robinanil : http://svn.apache.org/viewcvs.cgi/?root=Apache-SVN&view=rev&rev=1231700
Files : 
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/FPGrowthDriver.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/MultiTransactionTreeIterator.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowth.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/ParallelFPGrowthCombiner.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/ParallelFPGrowthMapper.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/ParallelFPGrowthReducer.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/TransactionSortingMapper.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/TransactionSortingReducer.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/TransactionTree.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/TransactionTreeIterator.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth/FrequentPatternMaxHeap.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth/Pattern.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth2
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth2/FPGrowthIds.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth2/FPGrowthObj.java
* /mahout/trunk/core/src/main/java/org/apache/mahout/fpm/pfpgrowth/fpgrowth2/FPTree.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/FPGrowthRetailDataTest2.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/FPGrowthRetailDataTestVs.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/FPGrowthSyntheticDataTest.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/FPGrowthTest2.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthRetailDataTest.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthRetailDataTest2.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthRetailDataTestVs.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthSynthDataTest2.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthTest.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/PFPGrowthTest2.java
* /mahout/trunk/core/src/test/java/org/apache/mahout/fpm/pfpgrowth/TransactionTreeTest.java
* /mahout/trunk/core/src/test/resources/FPGsynth.dat

                
> FPG uses a lot of boxed primitives - this patch eliminates a bunch of List<Integer> 
> ------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-921
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-921
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Frequent Itemset/Association Rule Mining
>    Affects Versions: 0.6
>            Reporter: tom pierce
>            Assignee: Robin Anil
>            Priority: Minor
>             Fix For: 0.6
>
>         Attachments: MAHOUT-921.patch
>
>
> TransactionTree uses List<Integer> internally and as part of its API; this patch changes that and pushes the use of List<Integer> down to FPGrowth.java.  This can be converted if the perf bug can be fixed (MAHOUT-890).  I will also provide an alternate base FPGrowth implementation.
> Please note that this patch assumes the patch attached to MAHOUT-920 has already been applied.

--
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] [Assigned] (MAHOUT-921) FPG uses a lot of boxed primitives - this patch eliminates a bunch of List

Posted by "Robin Anil (Assigned) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/MAHOUT-921?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Robin Anil reassigned MAHOUT-921:
---------------------------------

    Assignee: Robin Anil
    
> FPG uses a lot of boxed primitives - this patch eliminates a bunch of List<Integer> 
> ------------------------------------------------------------------------------------
>
>                 Key: MAHOUT-921
>                 URL: https://issues.apache.org/jira/browse/MAHOUT-921
>             Project: Mahout
>          Issue Type: Improvement
>          Components: Frequent Itemset/Association Rule Mining
>    Affects Versions: 0.6
>            Reporter: tom pierce
>            Assignee: Robin Anil
>            Priority: Minor
>         Attachments: MAHOUT-921.patch
>
>
> TransactionTree uses List<Integer> internally and as part of its API; this patch changes that and pushes the use of List<Integer> down to FPGrowth.java.  This can be converted if the perf bug can be fixed (MAHOUT-890).  I will also provide an alternate base FPGrowth implementation.
> Please note that this patch assumes the patch attached to MAHOUT-920 has already been applied.

--
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