You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mahout.apache.org by sr...@apache.org on 2009/08/21 15:29:47 UTC

svn commit: r806545 - /lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java

Author: srowen
Date: Fri Aug 21 13:29:47 2009
New Revision: 806545

URL: http://svn.apache.org/viewvc?rev=806545&view=rev
Log:
Fix a small-but-vital bug in the new JDBC data model implementation. Selecting user IDs in place of item IDs in one case. Oops.

Modified:
    lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java

Modified: lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java
URL: http://svn.apache.org/viewvc/lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java?rev=806545&r1=806544&r2=806545&view=diff
==============================================================================
--- lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java (original)
+++ lucene/mahout/trunk/core/src/main/java/org/apache/mahout/cf/taste/impl/model/jdbc/AbstractJDBCDataModel.java Fri Aug 21 13:29:47 2009
@@ -381,7 +381,7 @@
 
       FastIDSet result = new FastIDSet();
       while (rs.next()) {
-        result.add(rs.getLong(1));
+        result.add(rs.getLong(2));
       }
 
       if (result.isEmpty()) {