epugh commented on a change in pull request #2306:
URL: https://github.com/apache/lucene-solr/pull/2306#discussion_r571439997
##########
File path: solr/core/src/java/org/apache/solr/handler/UpdateRequestHandler.java
##########
@@ -134,12 +135,23 @@ protected void setAssumeContentType(String ct) {
}
private Map<String ,ContentStreamLoader> pathVsLoaders = new HashMap<>();
protected Map<String,ContentStreamLoader>
createDefaultLoaders(@SuppressWarnings({"rawtypes"})NamedList args) {
+
SolrParams p = null;
if(args!=null) {
p = args.toSolrParams();
}
Map<String,ContentStreamLoader> registry = new HashMap<>();
- registry.put("application/xml", new XMLLoader().init(p) );
+
+ // Try to load the scripting contrib modules XSLTLoader, and if not, fall
back to the XMLLoader.
+ // Enable the XSLTLoader by including the Scripting contrib module.
+ try {
+ final Class<?> clazz = Class.forName(
"org.apache.solr.scripting.util.xslt.XSLTLoader" );
Review comment:
This was the part where I didn't like what I did... And was putting
*something* up as a starter.. I really appreciate your and @uschindler input
on this. Definitily lets find another approach!
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
