[aperture-commit] SF.net SVN: aperture:[2495] aperture/trunk/example/src/main/java/org/ semanticdesktop/aperture/examples/handler/SimpleCrawlerHandler.java

[<my...@us...>]

Revision: 2495
          http://aperture.svn.sourceforge.net/aperture/?rev=2495&view=rev
Author:   mylka
Date:     2011-06-07 12:25:23 +0000 (Tue, 07 Jun 2011)

Log Message:
-----------
fixed an important bug in aperture-example. A model obtained from model set doesn't need to be opened the second time. It shouldn't make much difference, but in reality, the RepositoryModel implementation for RDF2Go had a bug. When you opened it twice, it would open two repository connections. The first one would be leaked. In cases where many models are created, this led to memory issues.

Modified Paths:
--------------
    aperture/trunk/example/src/main/java/org/semanticdesktop/aperture/examples/handler/SimpleCrawlerHandler.java

Modified: aperture/trunk/example/src/main/java/org/semanticdesktop/aperture/examples/handler/SimpleCrawlerHandler.java
===================================================================
--- aperture/trunk/example/src/main/java/org/semanticdesktop/aperture/examples/handler/SimpleCrawlerHandler.java	2011-05-31 14:19:24 UTC (rev 2494)
+++ aperture/trunk/example/src/main/java/org/semanticdesktop/aperture/examples/handler/SimpleCrawlerHandler.java	2011-06-07 12:25:23 UTC (rev 2495)
@@ -333,8 +333,12 @@
 
         // when running performance tests, we dump the dataobjects,
         // otherwise we channel the triples into the modelSet
-        Model model = (modelSet == null) ? RDF2Go.getModelFactory().createModel() : modelSet.getModel(uri);
-        model.open();
+        Model model = null;
+        if (modelSet == null) {
+            model = RDF2Go.getModelFactory().createModel().open(); 
+        } else {
+            model = modelSet.getModel(uri);
+        }
         return new RDFContainerImpl(model, uri);
     }
 


This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site.