Thats ok with me, but can we package them in data/ ? Hate to have an
external dependency for default behavior.

On Sunday, January 5, 2014, Matt Hayes wrote:

> In src/java/datafu/pig/text/SentenceDetect.java:
>
> > + * outfoo = FOREACH input GENERATE SentenceDetect(text) as sentences;
> > + * }
> > + * </pre>
> > + */
> > +public class SentenceDetect extends SimpleEvalFunc<DataBag>
> > +{
> > +    private static boolean isFirst = true;
> > +    InputStream is = null;
> > +    SentenceModel model = null;
> > +    SentenceDetectorME sdetector = null;
> > +    TupleFactory tf = TupleFactory.getInstance();
> > +    BagFactory bf = BagFactory.getInstance();
> > +
> > +    public DataBag call(String inputString) throws IOException
> > +    {
> > +         DataBag outBag = this.call(inputString, "data/en-sent.bin");
>
> Does it make sense to have a default model? Seems like this should be
> required by the constructor and then loaded into the distributed cache
> using getCacheFiles().
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/linkedin/datafu/pull/84/files#r8656855>
> .
>


-- 
Russell Jurney twitter.com/rjurney russell.jurney@gmail.com datasyndrome.com