I will do it. But I need to first get the simple test working in order to move forward. I hope I someone here can help me.
Thanks, Anuj Kumar On Tue, Feb 14, 2017 at 2:13 PM, Osma Suominen <osma.suomi...@helsinki.fi> wrote: > 14.02.2017, 15:03, anuj kumar kirjoitti: > >> I was working on a local copy of Jena source code initially. >> I have now forked Jena and added my specific files as I specified in my >> previous email to ease debugging by more experienced Jena developers. >> The forked repo can be found here : https://github.com/EaseTech/jena >> >> You will see that most of the code in these new files is simply the one >> that existed for Lucene based files. My first goal is toinstantiate the >> TextIndexES file and get the test case working. I will then move to >> implement the actual ES code, which IMO, should be much faster. >> > > Great, I hope you get this working! > > If you feel that you are duplicating existing Lucene code in your ES > implementation, consider abstracting that out into e.g. a common superclass > instead. This is something that already bothers me in the current Lucene vs > Solr implementations - there's even a "DRY" comment in the code showing > that somebody else has thought about it too. > > Also it might be helpful to try to reuse all the Lucene unit tests for ES > as well, if you can figure out a way to do that. > > -Osma > > > -- > Osma Suominen > D.Sc. (Tech), Information Systems Specialist > National Library of Finland > P.O. Box 26 (Kaikukatu 4) > 00014 HELSINGIN YLIOPISTO > Tel. +358 50 3199529 > osma.suomi...@helsinki.fi > http://www.nationallibrary.fi > -- *Anuj Kumar*
