Robert, the patch looks almost good. You should also change the pom.xml.template to reflect the new dependency. I'm still thinking about moving snowball into analyzers as a analyzers/snowball would that make sense?
Somewhat unrelated but still ugly:
Class<?> stemClass = Class.forName("org.tartarus.snowball.ext." + name + "Stemmer");
When I look through the patch I see this "name" parameter which is used to load a stemmer per reflection. We should really define a factory interface that creates the stemmer and get rid of the refelction code