You are viewing a plain text version of this content. The canonical link for it is here.
Posted to olio-commits@incubator.apache.org by sh...@apache.org on 2009/04/29 17:07:48 UTC

svn commit: r769831 - in /incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader: EventImages.java Images.java PersonImages.java

Author: shanti
Date: Wed Apr 29 17:07:48 2009
New Revision: 769831

URL: http://svn.apache.org/viewvc?rev=769831&view=rev
Log:
Fix for OLIO-86. The issue was that the same id was being used to represent
the key and filename in the images table. Since all images 
reside in the images table and the person images follow the thumbnail
images, the key needs to be offset appropriately so that the images table
has the correct filenames and id's for persons.

Modified:
    incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/EventImages.java
    incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/Images.java
    incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/PersonImages.java

Modified: incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/EventImages.java
URL: http://svn.apache.org/viewvc/incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/EventImages.java?rev=769831&r1=769830&r2=769831&view=diff
==============================================================================
--- incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/EventImages.java (original)
+++ incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/EventImages.java Wed Apr 29 17:07:48 2009
@@ -22,6 +22,7 @@
     public void prepare() {
      imageId = getSequence();
      ++imageId;
+	 imageKey = imageId;
      prefix = "e";
     }
 }

Modified: incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/Images.java
URL: http://svn.apache.org/viewvc/incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/Images.java?rev=769831&r1=769830&r2=769831&view=diff
==============================================================================
--- incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/Images.java (original)
+++ incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/Images.java Wed Apr 29 17:07:48 2009
@@ -42,7 +42,7 @@
 
     static Logger logger = Logger.getLogger(Comments.class.getName());
 
-    int imageId;
+    int imageKey, imageId;
     String prefix;
 
     public String getClearStatement() {
@@ -62,7 +62,7 @@
             s.setInt(4, 1280);
             s.setInt(5, 960);
             s.setString(6, prefix + imageId + "t.jpg");
-            s.setInt(7, this.imageId);
+            s.setInt(7, this.imageKey);
             c.addBatch();
         } catch (SQLException e) {
             logger.log(Level.SEVERE, e.getMessage(), e);

Modified: incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/PersonImages.java
URL: http://svn.apache.org/viewvc/incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/PersonImages.java?rev=769831&r1=769830&r2=769831&view=diff
==============================================================================
--- incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/PersonImages.java (original)
+++ incubator/olio/workload/rails/trunk/src/org/apache/olio/workload/loader/PersonImages.java Wed Apr 29 17:07:48 2009
@@ -25,7 +25,8 @@
     public void prepare() {
      int i = getSequence();
      ++i;
-     imageId = ScaleFactors.events + i;
+	 imageId = i;
+     imageKey = ScaleFactors.events + i;
      prefix = "p";
     }
 }