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()) {