You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@uima.apache.org by to...@apache.org on 2013/01/11 09:15:07 UTC
svn commit: r1431931 - in /uima/addons/trunk/Solrcas/src:
main/java/org/apache/uima/solrcas/ test/java/org/apache/uima/solrcas/
Author: tommaso
Date: Fri Jan 11 08:15:07 2013
New Revision: 1431931
URL: http://svn.apache.org/viewvc?rev=1431931&view=rev
Log:
[UIMA-2566] - replaced deprecated CommonsHttpSolrServer with HttpSolrServer plus minor improvements
Modified:
uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/FieldMappingReader.java
uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/SolrCASConsumer.java
uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/UriUtils.java
uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/EmbeddedSolrCASConsumer.java
uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/SolrCasConsumerIntegrationTest.java
Modified: uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/FieldMappingReader.java
URL: http://svn.apache.org/viewvc/uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/FieldMappingReader.java?rev=1431931&r1=1431930&r2=1431931&view=diff
==============================================================================
--- uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/FieldMappingReader.java (original)
+++ uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/FieldMappingReader.java Fri Jan 11 08:15:07 2013
@@ -52,9 +52,7 @@ public class FieldMappingReader {
input.close();
- SolrMappingConfiguration solrMappingConfiguration = new SolrMappingConfiguration(documentText, documentLanguage, fieldMapping);
-
- return solrMappingConfiguration;
+ return new SolrMappingConfiguration(documentText, documentLanguage, fieldMapping);
}
private static class FieldMappingHandler extends DefaultHandler {
Modified: uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/SolrCASConsumer.java
URL: http://svn.apache.org/viewvc/uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/SolrCASConsumer.java?rev=1431931&r1=1431930&r2=1431931&view=diff
==============================================================================
--- uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/SolrCASConsumer.java (original)
+++ uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/SolrCASConsumer.java Fri Jan 11 08:15:07 2013
@@ -23,14 +23,13 @@ import java.io.IOException;
import java.io.InputStream;
import java.net.URI;
import java.net.URISyntaxException;
-import java.net.URL;
import java.util.Map;
import javax.xml.parsers.ParserConfigurationException;
import org.apache.solr.client.solrj.SolrServer;
import org.apache.solr.client.solrj.SolrServerException;
-import org.apache.solr.client.solrj.impl.CommonsHttpSolrServer;
+import org.apache.solr.client.solrj.impl.HttpSolrServer;
import org.apache.solr.common.SolrInputDocument;
import org.apache.uima.UimaContext;
import org.apache.uima.analysis_component.CasAnnotator_ImplBase;
@@ -177,8 +176,7 @@ public class SolrCASConsumer extends Cas
InputStream input = getURI(mappingFileParam).toURL().openStream();
- SolrMappingConfiguration solrMappingConfiguration = fieldMappingReader.getConf(input);
- return solrMappingConfiguration;
+ return fieldMappingReader.getConf(input);
}
protected SolrServer createServer() throws SolrServerException {
@@ -193,8 +191,7 @@ public class SolrCASConsumer extends Cas
getConfigParameterValue("solrPath"));
if (solrInstanceTypeParam.equalsIgnoreCase("http")) {
- URL solrURL = UriUtils.create(solrPathParam).toURL();
- solrServer = new CommonsHttpSolrServer(solrURL);
+ solrServer = new HttpSolrServer(solrPathParam);
}
} catch (Exception e) {
throw new SolrServerException("Error creating SolrServer", e);
Modified: uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/UriUtils.java
URL: http://svn.apache.org/viewvc/uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/UriUtils.java?rev=1431931&r1=1431930&r2=1431931&view=diff
==============================================================================
--- uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/UriUtils.java (original)
+++ uima/addons/trunk/Solrcas/src/main/java/org/apache/uima/solrcas/UriUtils.java Fri Jan 11 08:15:07 2013
@@ -42,7 +42,7 @@ public class UriUtils {
* @return URI with proper quoting
* @throws URISyntaxException
*/
- public static URI quote (String s) throws URISyntaxException {
+ public static URI quote(String s) throws URISyntaxException {
if (-1 == s.indexOf('%')) {
// 3 argument constructor does any needed quoting of otherwise illegal chars
// https://issues.apache.org/jira/browse/UIMA-2097
@@ -69,7 +69,7 @@ public class UriUtils {
* Create a URI from a String, with proper quoting.
* Already quoted things in the input string are not re-quoted.
* Mimic exception treatment of URI.create
- * @param u
+ * @param s
* @return URI with proper quoting
*/
Modified: uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/EmbeddedSolrCASConsumer.java
URL: http://svn.apache.org/viewvc/uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/EmbeddedSolrCASConsumer.java?rev=1431931&r1=1431930&r2=1431931&view=diff
==============================================================================
--- uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/EmbeddedSolrCASConsumer.java (original)
+++ uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/EmbeddedSolrCASConsumer.java Fri Jan 11 08:15:07 2013
@@ -27,6 +27,10 @@ import org.apache.solr.client.solrj.Solr
import org.apache.solr.client.solrj.embedded.EmbeddedSolrServer;
import org.apache.solr.core.CoreContainer;
+/**
+ * Test {@link SolrCASConsumer} extension which uses and {@link EmbeddedSolrServer}
+ * underneath to communicate with Solr.
+ */
public class EmbeddedSolrCASConsumer extends SolrCASConsumer {
@Override
Modified: uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/SolrCasConsumerIntegrationTest.java
URL: http://svn.apache.org/viewvc/uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/SolrCasConsumerIntegrationTest.java?rev=1431931&r1=1431930&r2=1431931&view=diff
==============================================================================
--- uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/SolrCasConsumerIntegrationTest.java (original)
+++ uima/addons/trunk/Solrcas/src/test/java/org/apache/uima/solrcas/SolrCasConsumerIntegrationTest.java Fri Jan 11 08:15:07 2013
@@ -94,7 +94,7 @@ public class SolrCasConsumerIntegrationT
private CAS getCAS() throws ResourceInitializationException, CASException {
// Create an initial CASMgr from the factory.
CASMgr casMgr0 = CASFactory.createCAS();
- CASMgr casMgr = null;
+
// this call does nothing: because 2nd arg is null
CasCreationUtils.setupTypeSystem(casMgr0, null);
// Create a writable type system.
@@ -103,7 +103,7 @@ public class SolrCasConsumerIntegrationT
// Commit the type system.
((CASImpl) casMgr0).commitTypeSystem();
- casMgr = CASFactory.createCAS(tsa);
+ CASMgr casMgr = CASFactory.createCAS(tsa);
// Create the Base indexes.
casMgr.initCASIndexes();