Re: Merging database index with fulltext index
I would suggest you try LuSql, which was designed specifically to index relational databases into Lucene. It has an extensive user manual/tutorial which has some complex examples involving multi-joins and sub-queries. I am the author of LuSql. LuSql home page: http://lab.cisti-icist.nrc-cnrc.gc.ca/cistilabswiki/index.php/LuSql LuSql manual: http://cuvier.cisti.nrc.ca/~gnewton/lusql/v0.9/lusqlManual.pdf.html thanks, Glen 2009/2/28 : > Hi, > > what is the best approach to merge a database index with a lucene fulltext > index? Both databases store a unique ID per doc. This is the join criteria. > > requirements: > > * both resultsets may be very big (100.000 and much more) > * the merged resultset must be sorted by database index and/or relevance > * optional paging the merged resultset, a page has a size of 1000 docs max. > > example: > > select a, b from dbtable where c = 'foo' and content='bar' order by > relevance, a desc, d > > I would split this into: > > database: select ID, a, b from dbtable where c = 'foo' order by a desc, d > lucene: content:bar (sort:relevance) > merge: loop over the lucene resultset and add the db record into a new list > if the ID matches. > > If the resultset must be paged: > > database: select ID from dbtable where c = 'foo' order by a desc, d > lucene: content:bar (sort:relevance) > merge: loop over the lucene resultset and add the db record into a new list > if the ID matches. > page 1: select a,b from dbtable where ID IN (list of the ID's of page 1) > page 2: select a,b from dbtable where ID IN (list of the ID's of page 2) > ... > > > Is there a better way? > > Thank you. > > > > > > - > To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org > For additional commands, e-mail: java-user-h...@lucene.apache.org > > -- - - To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org For additional commands, e-mail: java-user-h...@lucene.apache.org
Re: queryNorm affect on score
FWIW, Hossman pointed out that the difference between index and query time boosts is that index time boosts on title, for instance, express "I care about this document's title more than other documents' titles [when it matches]" Query time boosts express "I care about matches on the title field more than matches on other fields". Best Erick On Sun, Mar 1, 2009 at 8:57 PM, Peter Keegan wrote: > As suggested, I added a query-time boost of 0.0f to the 'literals' field > (with index-time boost still there) and I did get the same scores for both > queries :) (there is a subtlety between index-time and query-time boosting > that I missed.) > > I also tried disabling the coord factor, but that had no affect on the > score, when combined with the above. This seems ok in this example since > the > the matching terms had boost = 0. > > Thanks Yonik, > Peter > > > > On Sat, Feb 28, 2009 at 6:02 PM, Yonik Seeley >wrote: > > > On Sat, Feb 28, 2009 at 3:02 PM, Peter Keegan > > wrote: > > >> in situations where you deal with simple query types, and matching > > query > > > structures, the queryNorm > > >> *can* be used to make scores semi-comparable. > > > > > > Hmm. My example used matching query structures. The only difference was > a > > > single term in a field with zero weight that didn't exist in the > matching > > > document. But one score was 3X the other. > > > > But the zero boost was an index-time boost, and the queryNorm takes > > into account query-time boosts and idfs. You might get closer to what > > you expect with a query time boost of 0.0f > > > > The other thing affecting the score is the coord factor - the fact > > that fewer of the optional terms matched (1/2) lowers the score. The > > coordination factor can be disabled on any BooleanQuery. > > > > If you do both of the above, I *think* you would get the same scores > > for this specific example. > > > > -Yonik > > http://www.lucidimagination.com > > > > - > > To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org > > For additional commands, e-mail: java-user-h...@lucene.apache.org > > > > >
Re: queryNorm affect on score
As suggested, I added a query-time boost of 0.0f to the 'literals' field (with index-time boost still there) and I did get the same scores for both queries :) (there is a subtlety between index-time and query-time boosting that I missed.) I also tried disabling the coord factor, but that had no affect on the score, when combined with the above. This seems ok in this example since the the matching terms had boost = 0. Thanks Yonik, Peter On Sat, Feb 28, 2009 at 6:02 PM, Yonik Seeley wrote: > On Sat, Feb 28, 2009 at 3:02 PM, Peter Keegan > wrote: > >> in situations where you deal with simple query types, and matching > query > > structures, the queryNorm > >> *can* be used to make scores semi-comparable. > > > > Hmm. My example used matching query structures. The only difference was a > > single term in a field with zero weight that didn't exist in the matching > > document. But one score was 3X the other. > > But the zero boost was an index-time boost, and the queryNorm takes > into account query-time boosts and idfs. You might get closer to what > you expect with a query time boost of 0.0f > > The other thing affecting the score is the coord factor - the fact > that fewer of the optional terms matched (1/2) lowers the score. The > coordination factor can be disabled on any BooleanQuery. > > If you do both of the above, I *think* you would get the same scores > for this specific example. > > -Yonik > http://www.lucidimagination.com > > - > To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org > For additional commands, e-mail: java-user-h...@lucene.apache.org > >
Re: Faceted Search using Lucene
You're calling get() too many times. For every call to get() you must match with a call to release(). So, once at the front of your search method you should: MultiSearcher searcher = get(); then use that searcher to do searching, retrieve docs, etc. Then in the finally clause, pass that searcher to release. So, only one call to get() and one matching call to release(). Mike Amin Mohammed-Coleman wrote: Hi The searchers are injected into the class via Spring. So when a client calls the class it is fully configured with a list of index searchers. However I have removed this list and instead injecting a list of directories which are passed to the DocumentSearchManager. DocumentSearchManager is SearchManager (should've mentioned that earlier). So finally I have modified by release code to do the following: private void release(MultiSearcher multiSeacher) throws Exception { IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSeacher.getSearchables(); for(int i =0 ; i < indexSearchers.length;i++) { documentSearcherManagers[i].release(indexSearchers[i]); } } and it's use looks like this: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '" + indexSearchers.size() +"'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = get().doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } } catch (Exception e) { throw new IllegalStateException(e); } finally { release(get()); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } So the final post construct constructs the DocumentSearchMangers with the list of directories..looking like this @PostConstruct public void initialiseDocumentSearcher() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); documentSearcherManagers = new DocumentSearcherManager[directories.size()]; for (int i = 0; i < directories.size() ;i++) { Directory directory = directories.get(i); DocumentSearcherManager documentSearcherManager = newDocumentSearcherManager(directory); documentSearcherManagers[i]=documentSearcherManager; } LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } Cheers Amin On Sun, Mar 1, 2009 at 6:15 PM, Michael McCandless < luc...@mikemccandless.com> wrote: I don't understand where searchers comes from, prior to initializeDocumentSearcher? You should, instead, simply create the SearcherManager (from your Directory instances). You don't need any searchers during initialize. Is DocumentSearcherManager the same as SearcherManager (just renamed)? The release method is wrong -- you're calling .get() and then immediately release. Instead, you should step through the searchers from your MultiSearcher and release them to each SearcherManager. You should call your release() in a finally clause. Mike Amin Mohammed-Coleman wrote: Sorry...i'm getting slightly confused. I have a PostConstruct which is where I should create an array of SearchManagers (per indexSeacher). From there I initialise the multisearcher using the get(). After which I need to call maybeReopen for each IndexSearcher. So I'll do the following: @PostConstruct public void initialiseDocumentSearcher() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrap
Re: Faceted Search using Lucene
Hi The searchers are injected into the class via Spring. So when a client calls the class it is fully configured with a list of index searchers. However I have removed this list and instead injecting a list of directories which are passed to the DocumentSearchManager. DocumentSearchManager is SearchManager (should've mentioned that earlier). So finally I have modified by release code to do the following: private void release(MultiSearcher multiSeacher) throws Exception { IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSeacher.getSearchables(); for(int i =0 ; i < indexSearchers.length;i++) { documentSearcherManagers[i].release(indexSearchers[i]); } } and it's use looks like this: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '" + indexSearchers.size() +"'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = get().doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } } catch (Exception e) { throw new IllegalStateException(e); } finally { release(get()); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } So the final post construct constructs the DocumentSearchMangers with the list of directories..looking like this @PostConstruct public void initialiseDocumentSearcher() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); documentSearcherManagers = new DocumentSearcherManager[directories.size()]; for (int i = 0; i < directories.size() ;i++) { Directory directory = directories.get(i); DocumentSearcherManager documentSearcherManager = newDocumentSearcherManager(directory); documentSearcherManagers[i]=documentSearcherManager; } LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } Cheers Amin On Sun, Mar 1, 2009 at 6:15 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > I don't understand where searchers comes from, prior to > initializeDocumentSearcher? You should, instead, simply create the > SearcherManager (from your Directory instances). You don't need any > searchers during initialize. > > Is DocumentSearcherManager the same as SearcherManager (just renamed)? > > The release method is wrong -- you're calling .get() and then > immediately release. Instead, you should step through the searchers > from your MultiSearcher and release them to each SearcherManager. > > You should call your release() in a finally clause. > > Mike > > Amin Mohammed-Coleman wrote: > > Sorry...i'm getting slightly confused. >> I have a PostConstruct which is where I should create an array of >> SearchManagers (per indexSeacher). From there I initialise the >> multisearcher using the get(). After which I need to call maybeReopen for >> each IndexSearcher. So I'll do the following: >> >> @PostConstruct >> >> public void initialiseDocumentSearcher() { >> >> PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( >> analyzer); >> >> analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), >> newKeywordAnalyzer()); >> >> queryParser = >> newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), >> analyzerWrapper); >> >> try { >> >> LOGGER.debug("Initialising multi searcher "); >> >> documentSearcherManagers = new DocumentSearcherManager[searchers.size()]; >> >> for
Re: Faceted Search using Lucene
I don't understand where searchers comes from, prior to initializeDocumentSearcher? You should, instead, simply create the SearcherManager (from your Directory instances). You don't need any searchers during initialize. Is DocumentSearcherManager the same as SearcherManager (just renamed)? The release method is wrong -- you're calling .get() and then immediately release. Instead, you should step through the searchers from your MultiSearcher and release them to each SearcherManager. You should call your release() in a finally clause. Mike Amin Mohammed-Coleman wrote: Sorry...i'm getting slightly confused. I have a PostConstruct which is where I should create an array of SearchManagers (per indexSeacher). From there I initialise the multisearcher using the get(). After which I need to call maybeReopen for each IndexSearcher. So I'll do the following: @PostConstruct public void initialiseDocumentSearcher() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); documentSearcherManagers = new DocumentSearcherManager[searchers.size()]; for (int i = 0; i < searchers.size() ;i++) { IndexSearcher indexSearcher = searchers.get(i); Directory directory = indexSearcher.getIndexReader().directory(); DocumentSearcherManager documentSearcherManager = newDocumentSearcherManager(directory); documentSearcherManagers[i]=documentSearcherManager; } LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } This initialises search managers. I then have methods: private void maybeReopen() throws Exception { LOGGER.debug("Initiating reopening of index readers..."); for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { documentSearcherManager.maybeReopen(); } } private void release() throws Exception { for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { documentSearcherManager.release(documentSearcherManager.get()); } } private MultiSearcher get() { List listOfIndexSeachers = new ArrayList(); for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { listOfIndexSeachers.add(documentSearcherManager.get()); } try { multiSearcher = new MultiSearcher(listOfIndexSeachers.toArray(newIndexSearcher[] {})); } catch (IOException e) { throw new IllegalStateException(e); } return multiSearcher; } These methods are used in the following manner in the search code: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '" + indexSearchers.size() +"'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = get().doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } release(); } catch (Exception e) { throw new IllegalStateException(e); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } Does this look better? Again..I really really appreciate your help! On Sun, Mar 1, 2009 at 4:18 PM, Michael McCandless < luc...@mikemccandless.com> wrote: This is not quite right -- you should only create SearcherManager once (per Direcotry) at startup/app load, not with every search request. And I don't see release -- it must call SearcherManager.release of each of the IndexSearchers previously returned from get(). Mike Amin Mohammed-Coleman wro
Re: Faceted Search using Lucene
Sorry...i'm getting slightly confused. I have a PostConstruct which is where I should create an array of SearchManagers (per indexSeacher). From there I initialise the multisearcher using the get(). After which I need to call maybeReopen for each IndexSearcher. So I'll do the following: @PostConstruct public void initialiseDocumentSearcher() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); documentSearcherManagers = new DocumentSearcherManager[searchers.size()]; for (int i = 0; i < searchers.size() ;i++) { IndexSearcher indexSearcher = searchers.get(i); Directory directory = indexSearcher.getIndexReader().directory(); DocumentSearcherManager documentSearcherManager = newDocumentSearcherManager(directory); documentSearcherManagers[i]=documentSearcherManager; } LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } This initialises search managers. I then have methods: private void maybeReopen() throws Exception { LOGGER.debug("Initiating reopening of index readers..."); for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { documentSearcherManager.maybeReopen(); } } private void release() throws Exception { for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { documentSearcherManager.release(documentSearcherManager.get()); } } private MultiSearcher get() { List listOfIndexSeachers = new ArrayList(); for (DocumentSearcherManager documentSearcherManager : documentSearcherManagers) { listOfIndexSeachers.add(documentSearcherManager.get()); } try { multiSearcher = new MultiSearcher(listOfIndexSeachers.toArray(newIndexSearcher[] {})); } catch (IOException e) { throw new IllegalStateException(e); } return multiSearcher; } These methods are used in the following manner in the search code: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '" + indexSearchers.size() +"'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = get().doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } release(); } catch (Exception e) { throw new IllegalStateException(e); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } Does this look better? Again..I really really appreciate your help! On Sun, Mar 1, 2009 at 4:18 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > This is not quite right -- you should only create SearcherManager once > (per Direcotry) at startup/app load, not with every search request. > > And I don't see release -- it must call SearcherManager.release of > each of the IndexSearchers previously returned from get(). > > Mike > > Amin Mohammed-Coleman wrote: > > Hi >> Thanks again for helping on a Sunday! >> >> I have now modified my maybeOpen() to do the following: >> >> private void maybeReopen() throws Exception { >> >> LOGGER.debug("Initiating reopening of index readers..."); >> >> IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSearcher >> .getSearchables(); >> >> for (IndexSearcher indexSearcher : indexSearchers) { >> >> IndexReader indexReader = indexSearcher.getIndexReader(); >> >> SearcherManager documentSearcherManager = new >> SearcherManager(indexReader.directory()); >> >> documentSearcherManager.maybeRe
Re: Faceted Search using Lucene
This is not quite right -- you should only create SearcherManager once (per Direcotry) at startup/app load, not with every search request. And I don't see release -- it must call SearcherManager.release of each of the IndexSearchers previously returned from get(). Mike Amin Mohammed-Coleman wrote: Hi Thanks again for helping on a Sunday! I have now modified my maybeOpen() to do the following: private void maybeReopen() throws Exception { LOGGER.debug("Initiating reopening of index readers..."); IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSearcher .getSearchables(); for (IndexSearcher indexSearcher : indexSearchers) { IndexReader indexReader = indexSearcher.getIndexReader(); SearcherManager documentSearcherManager = new SearcherManager(indexReader.directory()); documentSearcherManager.maybeReopen(); } } And get() to: private synchronized MultiSearcher get() { IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSearcher .getSearchables(); List indexSearchersList = new ArrayList(); for (IndexSearcher indexSearcher : indexSearchers) { IndexReader indexReader = indexSearcher.getIndexReader(); SearcherManager documentSearcherManager = null; try { documentSearcherManager = new SearcherManager(indexReader.directory()); } catch (IOException e) { throw new IllegalStateException(e); } indexSearchersList.add(documentSearcherManager.get()); } try { multiSearcher = new MultiSearcher(indexSearchersList.toArray(newIndexSearcher[] {})); } catch (IOException e) { throw new IllegalStateException(e); } return multiSearcher; } This makes all my test pass. I am using the SearchManager that you recommended. Does this look ok? On Sun, Mar 1, 2009 at 2:38 PM, Michael McCandless < luc...@mikemccandless.com> wrote: Your maybeReopen has an excess incRef(). I'm not sure how you open the searchers in the first place? The list starts as empty, and nothing populates it? When you do the initial population, you need an incRef. I think you're hitting IllegalStateException because maybeReopen is closing a reader before get() can get it (since they synchronize on different objects). I'd recommend switching to the SearcherManager class. Instantiate one for each of your searchers. On each search request, go through them and call maybeReopen(), and then call get() and gather each IndexSearcher instance into a new array. Then, make a new MultiSearcher (opposite of what I said before): while that creates a small amount of garbage, it'll keep your code simpler (good tradeoff). Mike Amin Mohammed-Coleman wrote: sorrry I added release(multiSearcher); instead of multiSearcher.close(); On Sun, Mar 1, 2009 at 2:17 PM, Amin Mohammed-Coleman wrote: Hi I've now done the following: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '"+ indexSearchers.size() + "'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = multiSearcher.doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } multiSearcher.close(); } catch (Exception e) { throw new IllegalStateException(e); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } And have the following methods: @PostConstruct public void initialiseQueryParser() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher .
Re: Faceted Search using Lucene
Hi Thanks again for helping on a Sunday! I have now modified my maybeOpen() to do the following: private void maybeReopen() throws Exception { LOGGER.debug("Initiating reopening of index readers..."); IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSearcher .getSearchables(); for (IndexSearcher indexSearcher : indexSearchers) { IndexReader indexReader = indexSearcher.getIndexReader(); SearcherManager documentSearcherManager = new SearcherManager(indexReader.directory()); documentSearcherManager.maybeReopen(); } } And get() to: private synchronized MultiSearcher get() { IndexSearcher[] indexSearchers = (IndexSearcher[]) multiSearcher .getSearchables(); List indexSearchersList = new ArrayList(); for (IndexSearcher indexSearcher : indexSearchers) { IndexReader indexReader = indexSearcher.getIndexReader(); SearcherManager documentSearcherManager = null; try { documentSearcherManager = new SearcherManager(indexReader.directory()); } catch (IOException e) { throw new IllegalStateException(e); } indexSearchersList.add(documentSearcherManager.get()); } try { multiSearcher = new MultiSearcher(indexSearchersList.toArray(newIndexSearcher[] {})); } catch (IOException e) { throw new IllegalStateException(e); } return multiSearcher; } This makes all my test pass. I am using the SearchManager that you recommended. Does this look ok? On Sun, Mar 1, 2009 at 2:38 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > Your maybeReopen has an excess incRef(). > > I'm not sure how you open the searchers in the first place? The list > starts as empty, and nothing populates it? > > When you do the initial population, you need an incRef. > > I think you're hitting IllegalStateException because maybeReopen is > closing a reader before get() can get it (since they synchronize on > different objects). > > I'd recommend switching to the SearcherManager class. Instantiate one > for each of your searchers. On each search request, go through them > and call maybeReopen(), and then call get() and gather each > IndexSearcher instance into a new array. Then, make a new > MultiSearcher (opposite of what I said before): while that creates a > small amount of garbage, it'll keep your code simpler (good > tradeoff). > > Mike > > Amin Mohammed-Coleman wrote: > > sorrry I added >> >> release(multiSearcher); >> >> >> instead of multiSearcher.close(); >> >> On Sun, Mar 1, 2009 at 2:17 PM, Amin Mohammed-Coleman > >wrote: >> >> Hi >>> I've now done the following: >>> >>> public Summary[] search(final SearchRequest searchRequest) >>> throwsSearchExecutionException { >>> >>> final String searchTerm = searchRequest.getSearchTerm(); >>> >>> if (StringUtils.isBlank(searchTerm)) { >>> >>> throw new SearchExecutionException("Search string cannot be empty. There >>> will be too many results to process."); >>> >>> } >>> >>> List summaryList = new ArrayList(); >>> >>> StopWatch stopWatch = new StopWatch("searchStopWatch"); >>> >>> stopWatch.start(); >>> >>> List indexSearchers = new ArrayList(); >>> >>> try { >>> >>> LOGGER.debug("Ensuring all index readers are up to date..."); >>> >>> maybeReopen(); >>> >>> LOGGER.debug("All Index Searchers are up to date. No of index searchers >>> '"+ indexSearchers.size() + >>> "'"); >>> >>> Query query = queryParser.parse(searchTerm); >>> >>> LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + >>> query.toString() +"'"); >>> >>> Sort sort = null; >>> >>> sort = applySortIfApplicable(searchRequest); >>> >>> Filter[] filters =applyFiltersIfApplicable(searchRequest); >>> >>> ChainedFilter chainedFilter = null; >>> >>> if (filters != null) { >>> >>> chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); >>> >>> } >>> >>> TopDocs topDocs = get().search(query,chainedFilter ,100,sort); >>> >>> ScoreDoc[] scoreDocs = topDocs.scoreDocs; >>> >>> LOGGER.debug("total number of hits for [" + query.toString() + " ] = >>> "+topDocs. >>> totalHits); >>> >>> for (ScoreDoc scoreDoc : scoreDocs) { >>> >>> final Document doc = multiSearcher.doc(scoreDoc.doc); >>> >>> float score = scoreDoc.score; >>> >>> final BaseDocument baseDocument = new BaseDocument(doc, score); >>> >>> Summary documentSummary = new DocumentSummaryImpl(baseDocument); >>> >>> summaryList.add(documentSummary); >>> >>> } >>> >>> multiSearcher.close(); >>> >>> } catch (Exception e) { >>> >>> throw new IllegalStateException(e); >>> >>> } >>> >>> stopWatch.stop(); >>> >>> LOGGER.debug("total time taken for document seach: " + >>> stopWatch.getTotalTimeMillis() + " ms"); >>> >>> return summaryList.toArray(new Summary[] {}); >>> >>> } >>> >>> >>> And have the following methods: >>> >>> @PostConstruct >>> >>> public void initialiseQueryParser() { >>> >>> PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( >>> analyzer); >>> >>> analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), >>> newKeywordAnalyzer()); >>> >>> queryParser = >>> newMultiFieldQueryPar
Re: Merging database index with fulltext index
I think the message is don't even try unless you're explored the alternatives and found them inadequate. Best Erick On Sun, Mar 1, 2009 at 2:19 AM, wrote: > > Yes. DBSight helps to flatten database objects into Lucene's > > documents. > > OK, thx for the advice. > > But back to my original question. > > When I have to merge both resultsets, what is the best approach to do this? > > > - > To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org > For additional commands, e-mail: java-user-h...@lucene.apache.org > >
Re: Faceted Search using Lucene
Hi again... Thanks for your patience, I modified the code to do the following: private void maybeReopen() throws Exception { startReopen(); try { MultiSearcher newMultiSeacher = get(); boolean refreshMultiSeacher = false; List indexSearchers = new ArrayList(); synchronized (searchers) { for (IndexSearcher indexSearcher: searchers) { IndexReader reader = indexSearcher.getIndexReader(); reader.incRef(); Directory directory = reader.directory(); long currentVersion = reader.getVersion(); if (IndexReader.getCurrentVersion(directory) != currentVersion) { IndexReader newReader = indexSearcher.getIndexReader().reopen(); if (newReader != reader) { reader.decRef(); refreshMultiSeacher = true; } reader = newReader; IndexSearcher newSearcher = new IndexSearcher(reader); indexSearchers.add(newSearcher); } } } if (refreshMultiSeacher) { try { newMultiSeacher = new MultiSearcher(indexSearchers.toArray(newIndexSearcher[] {})); warm(newMultiSeacher); swapMultiSearcher(newMultiSeacher); }finally { release(multiSearcher); } } } finally { doneReopen(); } } But I'm still getting an AlreadyCloseException this occurs when I call the get() method in the main search code. Cheers On Sun, Mar 1, 2009 at 2:24 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > OK new version of SearcherManager, that fixes maybeReopen() so that it can > be called from multiple threads. > > NOTE: it's still untested! > > Mike > > package lia.admin; > > import java.io.IOException; > import java.util.HashMap; > > import org.apache.lucene.search.IndexSearcher; > import org.apache.lucene.index.IndexReader; > import org.apache.lucene.store.Directory; > > /** Utility class to get/refresh searchers when you are > * using multiple threads. */ > > public class SearcherManager { > > private IndexSearcher currentSearcher; //A > private Directory dir; > > public SearcherManager(Directory dir) throws IOException { >this.dir = dir; >currentSearcher = new IndexSearcher(IndexReader.open(dir)); //B > } > > public void warm(IndexSearcher searcher) {}//C > > private boolean reopening; > > private synchronized void startReopen()//D >throws InterruptedException { >while (reopening) { > wait(); >} >reopening = true; > } > > private synchronized void doneReopen() { //E >reopening = false; >notifyAll(); > } > > public void maybeReopen() throws InterruptedException, IOException { //F > >startReopen(); > >try { > final IndexSearcher searcher = get(); > try { >long currentVersion = currentSearcher.getIndexReader().getVersion(); > //G >if (IndexReader.getCurrentVersion(dir) != currentVersion) { > //G > IndexReader newReader = currentSearcher.getIndexReader().reopen(); > //G > assert newReader != currentSearcher.getIndexReader(); > //G > IndexSearcher newSearcher = new IndexSearcher(newReader); > //G > warm(newSearcher); > //G > swapSearcher(newSearcher); > //G >} > } finally { >release(searcher); > } >} finally { > doneReopen(); >} > } > > public synchronized IndexSearcher get() { //H >currentSearcher.getIndexReader().incRef(); >return currentSearcher; > } > > public synchronized void release(IndexSearcher searcher) //I >throws IOException { >searcher.getIndexReader().decRef(); > } > > private synchronized void swapSearcher(IndexSearcher newSearcher) //J > throws IOException { >release(currentSearcher); >currentSearcher = newSearcher; > } > } > > /* > #A Current IndexSearcher > #B Create initial searcher > #C Implement in subclass to warm new searcher > #D Pauses until no other thread is reopening > #E Finish reopen and notify other threads > #F Reopen searcher if there are changes > #G Check index version and reopen, warm, swap if needed > #H Returns current searcher > #I Release searcher > #J Swaps currentSearcher to new searcher > */ > > Mike > > > On Mar 1, 2009, at 8:27 AM, Amin Mohammed-Coleman wrote: > > just a quick point: >> public void maybeReopen() throws IOException { //D >> long currentVersion = currentSearcher.getIndexReader().getVersion(); >> if (IndexReader.getCurrentVersion(dir) != currentVersion) { >>IndexReader newReader = currentSearcher.getIndexReader().reopen(); >>assert newReader != currentSearcher.getIndexReader(); >>IndexSearcher newSearcher = new IndexSearcher(newReader); >>warm(newSearcher); >>swapSearcher(newSearcher); >> } >> } >> >> should the above be synchronised? >> >> On Sun, Mar 1, 2009 at 1:25 PM, Amin Mohammed-Coleman > >wrote: >> >> thanks. i will rewrite..in between giving my baby her feed and playing >>> with the other child and my wife who wants me to do several other things! >>> >>> >>
Re: Faceted Search using Lucene
Your maybeReopen has an excess incRef(). I'm not sure how you open the searchers in the first place? The list starts as empty, and nothing populates it? When you do the initial population, you need an incRef. I think you're hitting IllegalStateException because maybeReopen is closing a reader before get() can get it (since they synchronize on different objects). I'd recommend switching to the SearcherManager class. Instantiate one for each of your searchers. On each search request, go through them and call maybeReopen(), and then call get() and gather each IndexSearcher instance into a new array. Then, make a new MultiSearcher (opposite of what I said before): while that creates a small amount of garbage, it'll keep your code simpler (good tradeoff). Mike Amin Mohammed-Coleman wrote: sorrry I added release(multiSearcher); instead of multiSearcher.close(); On Sun, Mar 1, 2009 at 2:17 PM, Amin Mohammed-Coleman >wrote: Hi I've now done the following: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '"+ indexSearchers.size() + "'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = multiSearcher.doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } multiSearcher.close(); } catch (Exception e) { throw new IllegalStateException(e); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } And have the following methods: @PostConstruct public void initialiseQueryParser() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); this.multiSearcher = new MultiSearcher(searchers.toArray(newIndexSearcher[] {})); LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } Initialises mutltisearcher when this class is creared by spring. private synchronized void swapMultiSearcher(MultiSearcher newMultiSearcher) { try { release(multiSearcher); } catch (IOException e) { throw new IllegalStateException(e); } multiSearcher = newMultiSearcher; } public void maybeReopen() throws IOException { MultiSearcher newMultiSeacher = null; boolean refreshMultiSeacher = false; List indexSearchers = new ArrayList(); synchronized (searchers) { for (IndexSearcher indexSearcher: searchers) { IndexReader reader = indexSearcher.getIndexReader(); reader.incRef(); Directory directory = reader.directory(); long currentVersion = reader.getVersion(); if (IndexReader.getCurrentVersion(directory) != currentVersion) { IndexReader newReader = indexSearcher.getIndexReader().reopen(); if (newReader != reader) { reader.decRef(); refreshMultiSeacher = true; } reader = newReader; IndexSearcher newSearcher = new IndexSearcher(newReader); indexSearchers.add(newSearcher); } } } if (refreshMultiSeacher) { newMultiSeacher = new MultiSearcher(indexSearchers.toArray(newIndexSearcher[] {})); warm(newMultiSeacher); swapMultiSearcher(newMultiSeacher); } } private void warm(MultiSearcher newMultiSeacher) { } private synchronized MultiSearcher get() { for (IndexSearcher indexSearcher: searchers) { indexSearcher.getIndexReader().incRef(); } return multiSearcher; } private synchronized void release(MultiSearcher multiSearcher) throwsIOException { for (IndexSearcher indexSearcher: searchers) { indexSearcher.ge
Re: Faceted Search using Lucene
OK new version of SearcherManager, that fixes maybeReopen() so that it can be called from multiple threads. NOTE: it's still untested! Mike package lia.admin; import java.io.IOException; import java.util.HashMap; import org.apache.lucene.search.IndexSearcher; import org.apache.lucene.index.IndexReader; import org.apache.lucene.store.Directory; /** Utility class to get/refresh searchers when you are * using multiple threads. */ public class SearcherManager { private IndexSearcher currentSearcher; //A private Directory dir; public SearcherManager(Directory dir) throws IOException { this.dir = dir; currentSearcher = new IndexSearcher(IndexReader.open(dir)); //B } public void warm(IndexSearcher searcher) {}//C private boolean reopening; private synchronized void startReopen()//D throws InterruptedException { while (reopening) { wait(); } reopening = true; } private synchronized void doneReopen() { //E reopening = false; notifyAll(); } public void maybeReopen() throws InterruptedException, IOException { //F startReopen(); try { final IndexSearcher searcher = get(); try { long currentVersion = currentSearcher.getIndexReader().getVersion(); //G if (IndexReader.getCurrentVersion(dir) != currentVersion) { //G IndexReader newReader = currentSearcher.getIndexReader().reopen(); //G assert newReader != currentSearcher.getIndexReader(); //G IndexSearcher newSearcher = new IndexSearcher(newReader); //G warm(newSearcher); //G swapSearcher(newSearcher); //G } } finally { release(searcher); } } finally { doneReopen(); } } public synchronized IndexSearcher get() { //H currentSearcher.getIndexReader().incRef(); return currentSearcher; } public synchronized void release(IndexSearcher searcher) //I throws IOException { searcher.getIndexReader().decRef(); } private synchronized void swapSearcher(IndexSearcher newSearcher) //J throws IOException { release(currentSearcher); currentSearcher = newSearcher; } } /* #A Current IndexSearcher #B Create initial searcher #C Implement in subclass to warm new searcher #D Pauses until no other thread is reopening #E Finish reopen and notify other threads #F Reopen searcher if there are changes #G Check index version and reopen, warm, swap if needed #H Returns current searcher #I Release searcher #J Swaps currentSearcher to new searcher */ Mike On Mar 1, 2009, at 8:27 AM, Amin Mohammed-Coleman wrote: just a quick point: public void maybeReopen() throws IOException { //D long currentVersion = currentSearcher.getIndexReader().getVersion(); if (IndexReader.getCurrentVersion(dir) != currentVersion) { IndexReader newReader = currentSearcher.getIndexReader().reopen(); assert newReader != currentSearcher.getIndexReader(); IndexSearcher newSearcher = new IndexSearcher(newReader); warm(newSearcher); swapSearcher(newSearcher); } } should the above be synchronised? On Sun, Mar 1, 2009 at 1:25 PM, Amin Mohammed-Coleman >wrote: thanks. i will rewrite..in between giving my baby her feed and playing with the other child and my wife who wants me to do several other things! On Sun, Mar 1, 2009 at 1:20 PM, Michael McCandless < luc...@mikemccandless.com> wrote: Amin Mohammed-Coleman wrote: Hi Thanks for your input. I would like to have a go at doing this myself first, Solr may be an option. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. -- I can moved the code out so that it is only created once and reused. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. -- This was something I thought about. I could move it out so that it's created once. However I presume inside my code i need to check whether the indexreaders are update to date. This needs to be synchronized as well I guess(?) Yes you should synchronize the check for whether the IndexReader is current. * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. -- So i need to extract the logic for reopening and provide a synchronisation mechanism. Yes. Ok. So I have some work to do. I'll refactor the code and see if I can get inline to your recommendations. On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandles
Re: Faceted Search using Lucene
sorrry I added release(multiSearcher); instead of multiSearcher.close(); On Sun, Mar 1, 2009 at 2:17 PM, Amin Mohammed-Coleman wrote: > Hi > I've now done the following: > > public Summary[] search(final SearchRequest searchRequest) > throwsSearchExecutionException { > > final String searchTerm = searchRequest.getSearchTerm(); > > if (StringUtils.isBlank(searchTerm)) { > > throw new SearchExecutionException("Search string cannot be empty. There > will be too many results to process."); > > } > > List summaryList = new ArrayList(); > > StopWatch stopWatch = new StopWatch("searchStopWatch"); > > stopWatch.start(); > > List indexSearchers = new ArrayList(); > > try { > > LOGGER.debug("Ensuring all index readers are up to date..."); > > maybeReopen(); > > LOGGER.debug("All Index Searchers are up to date. No of index searchers '"+ > indexSearchers.size() + > "'"); > > Query query = queryParser.parse(searchTerm); > > LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + > query.toString() +"'"); > > Sort sort = null; > > sort = applySortIfApplicable(searchRequest); > > Filter[] filters =applyFiltersIfApplicable(searchRequest); > > ChainedFilter chainedFilter = null; > > if (filters != null) { > > chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); > > } > > TopDocs topDocs = get().search(query,chainedFilter ,100,sort); > > ScoreDoc[] scoreDocs = topDocs.scoreDocs; > > LOGGER.debug("total number of hits for [" + query.toString() + " ] = > "+topDocs. > totalHits); > > for (ScoreDoc scoreDoc : scoreDocs) { > > final Document doc = multiSearcher.doc(scoreDoc.doc); > > float score = scoreDoc.score; > > final BaseDocument baseDocument = new BaseDocument(doc, score); > > Summary documentSummary = new DocumentSummaryImpl(baseDocument); > > summaryList.add(documentSummary); > > } > > multiSearcher.close(); > > } catch (Exception e) { > > throw new IllegalStateException(e); > > } > > stopWatch.stop(); > > LOGGER.debug("total time taken for document seach: " + > stopWatch.getTotalTimeMillis() + " ms"); > > return summaryList.toArray(new Summary[] {}); > > } > > > And have the following methods: > > @PostConstruct > > public void initialiseQueryParser() { > > PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( > analyzer); > > analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), > newKeywordAnalyzer()); > > queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), > analyzerWrapper); > > try { > > LOGGER.debug("Initialising multi searcher "); > > this.multiSearcher = new MultiSearcher(searchers.toArray(newIndexSearcher[] > {})); > > LOGGER.debug("multi searcher initialised"); > > } catch (IOException e) { > > throw new IllegalStateException(e); > > } > > } > > > Initialises mutltisearcher when this class is creared by spring. > > > private synchronized void swapMultiSearcher(MultiSearcher > newMultiSearcher) { > > try { > > release(multiSearcher); > > } catch (IOException e) { > > throw new IllegalStateException(e); > > } > > multiSearcher = newMultiSearcher; > > } > > public void maybeReopen() throws IOException { > > MultiSearcher newMultiSeacher = null; > > boolean refreshMultiSeacher = false; > > List indexSearchers = new ArrayList(); > > synchronized (searchers) { > > for (IndexSearcher indexSearcher: searchers) { > > IndexReader reader = indexSearcher.getIndexReader(); > > reader.incRef(); > > Directory directory = reader.directory(); > > long currentVersion = reader.getVersion(); > > if (IndexReader.getCurrentVersion(directory) != currentVersion) { > > IndexReader newReader = indexSearcher.getIndexReader().reopen(); > > if (newReader != reader) { > > reader.decRef(); > > refreshMultiSeacher = true; > > } > > reader = newReader; > > IndexSearcher newSearcher = new IndexSearcher(newReader); > > indexSearchers.add(newSearcher); > > } > > } > > } > > > > if (refreshMultiSeacher) { > > newMultiSeacher = new MultiSearcher(indexSearchers.toArray(newIndexSearcher[] > {})); > > warm(newMultiSeacher); > > swapMultiSearcher(newMultiSeacher); > > } > > > > } > > > private void warm(MultiSearcher newMultiSeacher) { > > } > > > > private synchronized MultiSearcher get() { > > for (IndexSearcher indexSearcher: searchers) { > > indexSearcher.getIndexReader().incRef(); > > } > > return multiSearcher; > > } > > private synchronized void release(MultiSearcher multiSearcher) > throwsIOException { > > for (IndexSearcher indexSearcher: searchers) { > > indexSearcher.getIndexReader().decRef(); > > } > > } > > > However I am now getting > > > java.lang.IllegalStateException: > org.apache.lucene.store.AlreadyClosedException: this IndexReader is closed > > > on the call: > > > private synchronized MultiSearcher get() { > > for (IndexSearcher indexSearcher: searchers) { > > indexSearcher.getIndexReader().incRef(); > > } > > return multiSearcher; > > } > > > I'm doing something wrong ..obviously..not sure where
Re: Faceted Search using Lucene
Hi I've now done the following: public Summary[] search(final SearchRequest searchRequest) throwsSearchExecutionException { final String searchTerm = searchRequest.getSearchTerm(); if (StringUtils.isBlank(searchTerm)) { throw new SearchExecutionException("Search string cannot be empty. There will be too many results to process."); } List summaryList = new ArrayList(); StopWatch stopWatch = new StopWatch("searchStopWatch"); stopWatch.start(); List indexSearchers = new ArrayList(); try { LOGGER.debug("Ensuring all index readers are up to date..."); maybeReopen(); LOGGER.debug("All Index Searchers are up to date. No of index searchers '" + indexSearchers.size() +"'"); Query query = queryParser.parse(searchTerm); LOGGER.debug("Search Term '" + searchTerm +"' > Lucene Query '" + query.toString() +"'"); Sort sort = null; sort = applySortIfApplicable(searchRequest); Filter[] filters =applyFiltersIfApplicable(searchRequest); ChainedFilter chainedFilter = null; if (filters != null) { chainedFilter = new ChainedFilter(filters, ChainedFilter.OR); } TopDocs topDocs = get().search(query,chainedFilter ,100,sort); ScoreDoc[] scoreDocs = topDocs.scoreDocs; LOGGER.debug("total number of hits for [" + query.toString() + " ] = "+topDocs. totalHits); for (ScoreDoc scoreDoc : scoreDocs) { final Document doc = multiSearcher.doc(scoreDoc.doc); float score = scoreDoc.score; final BaseDocument baseDocument = new BaseDocument(doc, score); Summary documentSummary = new DocumentSummaryImpl(baseDocument); summaryList.add(documentSummary); } multiSearcher.close(); } catch (Exception e) { throw new IllegalStateException(e); } stopWatch.stop(); LOGGER.debug("total time taken for document seach: " + stopWatch.getTotalTimeMillis() + " ms"); return summaryList.toArray(new Summary[] {}); } And have the following methods: @PostConstruct public void initialiseQueryParser() { PerFieldAnalyzerWrapper analyzerWrapper = new PerFieldAnalyzerWrapper( analyzer); analyzerWrapper.addAnalyzer(FieldNameEnum.TYPE.getDescription(), newKeywordAnalyzer()); queryParser = newMultiFieldQueryParser(FieldNameEnum.fieldNameDescriptions(), analyzerWrapper); try { LOGGER.debug("Initialising multi searcher "); this.multiSearcher = new MultiSearcher(searchers.toArray(new IndexSearcher[] {})); LOGGER.debug("multi searcher initialised"); } catch (IOException e) { throw new IllegalStateException(e); } } Initialises mutltisearcher when this class is creared by spring. private synchronized void swapMultiSearcher(MultiSearcher newMultiSearcher) { try { release(multiSearcher); } catch (IOException e) { throw new IllegalStateException(e); } multiSearcher = newMultiSearcher; } public void maybeReopen() throws IOException { MultiSearcher newMultiSeacher = null; boolean refreshMultiSeacher = false; List indexSearchers = new ArrayList(); synchronized (searchers) { for (IndexSearcher indexSearcher: searchers) { IndexReader reader = indexSearcher.getIndexReader(); reader.incRef(); Directory directory = reader.directory(); long currentVersion = reader.getVersion(); if (IndexReader.getCurrentVersion(directory) != currentVersion) { IndexReader newReader = indexSearcher.getIndexReader().reopen(); if (newReader != reader) { reader.decRef(); refreshMultiSeacher = true; } reader = newReader; IndexSearcher newSearcher = new IndexSearcher(newReader); indexSearchers.add(newSearcher); } } } if (refreshMultiSeacher) { newMultiSeacher = new MultiSearcher(indexSearchers.toArray(newIndexSearcher[] {})); warm(newMultiSeacher); swapMultiSearcher(newMultiSeacher); } } private void warm(MultiSearcher newMultiSeacher) { } private synchronized MultiSearcher get() { for (IndexSearcher indexSearcher: searchers) { indexSearcher.getIndexReader().incRef(); } return multiSearcher; } private synchronized void release(MultiSearcher multiSearcher) throwsIOException { for (IndexSearcher indexSearcher: searchers) { indexSearcher.getIndexReader().decRef(); } } However I am now getting java.lang.IllegalStateException: org.apache.lucene.store.AlreadyClosedException: this IndexReader is closed on the call: private synchronized MultiSearcher get() { for (IndexSearcher indexSearcher: searchers) { indexSearcher.getIndexReader().incRef(); } return multiSearcher; } I'm doing something wrong ..obviously..not sure where though.. Cheers On Sun, Mar 1, 2009 at 1:36 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > I was wondering the same thing ;) > > It's best to call this method from a single BG "warming" thread, in which > case it would not need its own synchronization. > > But, to be safe, I'll add internal synchronization to it. You can't simply > put synchronized in front of the method, since you don't want this to block > searching. > > > Mike > > Amin Mohammed-Coleman wrote: > > just a quick point: >> public void maybeReopen(
Re: Faceted Search using Lucene
I was wondering the same thing ;) It's best to call this method from a single BG "warming" thread, in which case it would not need its own synchronization. But, to be safe, I'll add internal synchronization to it. You can't simply put synchronized in front of the method, since you don't want this to block searching. Mike Amin Mohammed-Coleman wrote: just a quick point: public void maybeReopen() throws IOException { //D long currentVersion = currentSearcher.getIndexReader().getVersion(); if (IndexReader.getCurrentVersion(dir) != currentVersion) { IndexReader newReader = currentSearcher.getIndexReader().reopen(); assert newReader != currentSearcher.getIndexReader(); IndexSearcher newSearcher = new IndexSearcher(newReader); warm(newSearcher); swapSearcher(newSearcher); } } should the above be synchronised? On Sun, Mar 1, 2009 at 1:25 PM, Amin Mohammed-Coleman >wrote: thanks. i will rewrite..in between giving my baby her feed and playing with the other child and my wife who wants me to do several other things! On Sun, Mar 1, 2009 at 1:20 PM, Michael McCandless < luc...@mikemccandless.com> wrote: Amin Mohammed-Coleman wrote: Hi Thanks for your input. I would like to have a go at doing this myself first, Solr may be an option. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. -- I can moved the code out so that it is only created once and reused. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. -- This was something I thought about. I could move it out so that it's created once. However I presume inside my code i need to check whether the indexreaders are update to date. This needs to be synchronized as well I guess(?) Yes you should synchronize the check for whether the IndexReader is current. * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. -- So i need to extract the logic for reopening and provide a synchronisation mechanism. Yes. Ok. So I have some work to do. I'll refactor the code and see if I can get inline to your recommendations. On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandless < luc...@mikemccandless.com> wrote: On a quick look, I think there are a few problems with the code: * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. * You are over-incRef'ing (the reader.incRef inside the loop) -- I don't see a corresponding decRef. * You reopen and warm your searchers "live" (vs with BG thread); meaning the unlucky search request that hits a reopen pays the cost. This might be OK if the index is small enough that reopening & warming takes very little time. But if index gets large, making a random search pay that warming cost is not nice to the end user. It erodes their trust in you. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. You should consider simply using Solr -- it handles all this logic for you and has been well debugged with time... Mike Amin Mohammed-Coleman wrote: The reason for the indexreader.reopen is because I have a webapp which enables users to upload files and then search for the documents. If I don't reopen i'm concerned that the facet hit counter won't be updated. On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman < ami...@gmail.com wrote: Hi I have been able to get the code working for my scenario, however I have a question and I was wondering if I could get some help. I have a list of IndexSearchers which are used in a MultiSearcher class. I use the indexsearchers to get each indexreader and put them into a MultiIndexReader. IndexReader[] readers = new IndexReader[searchables.length]; for (int i =0 ; i < searchables.length;i++) { IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; readers[i] = indexSearcher.getIndexReader(); IndexReader newReader = readers[i].reopen(); if (newReader != readers[i]) { readers[i].close(); } readers[i] = newReader; } multiReader = new MultiReader(readers); OpenBitSetFacetHitCounter facetHitCounter = newOpenBitSetFacetHitCounter(); IndexSearcher indexSearcher = new IndexSearcher(multiReader); I then use the indexseacher to do the facet stuff. I end the code with closing the multireader. This is causing problems in another
Re: Faceted Search using Lucene
just a quick point: public void maybeReopen() throws IOException { //D long currentVersion = currentSearcher.getIndexReader().getVersion(); if (IndexReader.getCurrentVersion(dir) != currentVersion) { IndexReader newReader = currentSearcher.getIndexReader().reopen(); assert newReader != currentSearcher.getIndexReader(); IndexSearcher newSearcher = new IndexSearcher(newReader); warm(newSearcher); swapSearcher(newSearcher); } } should the above be synchronised? On Sun, Mar 1, 2009 at 1:25 PM, Amin Mohammed-Coleman wrote: > thanks. i will rewrite..in between giving my baby her feed and playing > with the other child and my wife who wants me to do several other things! > > > > On Sun, Mar 1, 2009 at 1:20 PM, Michael McCandless < > luc...@mikemccandless.com> wrote: > >> >> Amin Mohammed-Coleman wrote: >> >> Hi >>> Thanks for your input. I would like to have a go at doing this myself >>> first, Solr may be an option. >>> >>> * You are creating a new Analyzer & QueryParser every time, also >>> creating unnecessary garbage; instead, they should be created once >>> & reused. >>> >>> -- I can moved the code out so that it is only created once and reused. >>> >>> >>> * You always make a new IndexSearcher and a new MultiSearcher even >>> when nothing has changed. This just generates unnecessary garbage >>> which GC then must sweep up. >>> >>> -- This was something I thought about. I could move it out so that it's >>> created once. However I presume inside my code i need to check whether >>> the >>> indexreaders are update to date. This needs to be synchronized as well I >>> guess(?) >>> >> >> Yes you should synchronize the check for whether the IndexReader is >> current. >> >> * I don't see any synchronization -- it looks like two search >>> requests are allowed into this method at the same time? Which is >>> dangerous... eg both (or, more) will wastefully reopen the >>> readers. >>> -- So i need to extract the logic for reopening and provide a >>> synchronisation mechanism. >>> >> >> Yes. >> >> >> Ok. So I have some work to do. I'll refactor the code and see if I can >>> get >>> inline to your recommendations. >>> >>> >>> On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandless < >>> luc...@mikemccandless.com> wrote: >>> >>> On a quick look, I think there are a few problems with the code: * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. * You are over-incRef'ing (the reader.incRef inside the loop) -- I don't see a corresponding decRef. * You reopen and warm your searchers "live" (vs with BG thread); meaning the unlucky search request that hits a reopen pays the cost. This might be OK if the index is small enough that reopening & warming takes very little time. But if index gets large, making a random search pay that warming cost is not nice to the end user. It erodes their trust in you. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. You should consider simply using Solr -- it handles all this logic for you and has been well debugged with time... Mike Amin Mohammed-Coleman wrote: The reason for the indexreader.reopen is because I have a webapp which > enables users to upload files and then search for the documents. If I > don't > reopen i'm concerned that the facet hit counter won't be updated. > > On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman < > ami...@gmail.com > >> wrote: >> > > Hi > >> I have been able to get the code working for my scenario, however I >> have >> a >> question and I was wondering if I could get some help. I have a list >> of >> IndexSearchers which are used in a MultiSearcher class. I use the >> indexsearchers to get each indexreader and put them into a >> MultiIndexReader. >> >> IndexReader[] readers = new IndexReader[searchables.length]; >> >> for (int i =0 ; i < searchables.length;i++) { >> >> IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; >> >> readers[i] = indexSearcher.getIndexReader(); >> >> IndexReader newReader = readers[i].reopen(); >> >> if (newReader != readers[i]) { >> >> readers[i].close(); >> >> } >> >> readers[i] = newReader; >> >> >> >> } >> >> multiReader = new MultiReader(readers); >> >> OpenBitSetFacetHitCounter facetHitCo
Re: Faceted Search using Lucene
thanks. i will rewrite..in between giving my baby her feed and playing with the other child and my wife who wants me to do several other things! On Sun, Mar 1, 2009 at 1:20 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > Amin Mohammed-Coleman wrote: > > Hi >> Thanks for your input. I would like to have a go at doing this myself >> first, Solr may be an option. >> >> * You are creating a new Analyzer & QueryParser every time, also >> creating unnecessary garbage; instead, they should be created once >> & reused. >> >> -- I can moved the code out so that it is only created once and reused. >> >> >> * You always make a new IndexSearcher and a new MultiSearcher even >> when nothing has changed. This just generates unnecessary garbage >> which GC then must sweep up. >> >> -- This was something I thought about. I could move it out so that it's >> created once. However I presume inside my code i need to check whether >> the >> indexreaders are update to date. This needs to be synchronized as well I >> guess(?) >> > > Yes you should synchronize the check for whether the IndexReader is > current. > > * I don't see any synchronization -- it looks like two search >> requests are allowed into this method at the same time? Which is >> dangerous... eg both (or, more) will wastefully reopen the >> readers. >> -- So i need to extract the logic for reopening and provide a >> synchronisation mechanism. >> > > Yes. > > > Ok. So I have some work to do. I'll refactor the code and see if I can >> get >> inline to your recommendations. >> >> >> On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandless < >> luc...@mikemccandless.com> wrote: >> >> >>> On a quick look, I think there are a few problems with the code: >>> >>> * I don't see any synchronization -- it looks like two search >>> requests are allowed into this method at the same time? Which is >>> dangerous... eg both (or, more) will wastefully reopen the >>> readers. >>> >>> * You are over-incRef'ing (the reader.incRef inside the loop) -- I >>> don't see a corresponding decRef. >>> >>> * You reopen and warm your searchers "live" (vs with BG thread); >>> meaning the unlucky search request that hits a reopen pays the >>> cost. This might be OK if the index is small enough that >>> reopening & warming takes very little time. But if index gets >>> large, making a random search pay that warming cost is not nice to >>> the end user. It erodes their trust in you. >>> >>> * You always make a new IndexSearcher and a new MultiSearcher even >>> when nothing has changed. This just generates unnecessary garbage >>> which GC then must sweep up. >>> >>> * You are creating a new Analyzer & QueryParser every time, also >>> creating unnecessary garbage; instead, they should be created once >>> & reused. >>> >>> You should consider simply using Solr -- it handles all this logic for >>> you and has been well debugged with time... >>> >>> Mike >>> >>> Amin Mohammed-Coleman wrote: >>> >>> The reason for the indexreader.reopen is because I have a webapp which >>> enables users to upload files and then search for the documents. If I don't reopen i'm concerned that the facet hit counter won't be updated. On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman < ami...@gmail.com > wrote: > Hi > I have been able to get the code working for my scenario, however I > have > a > question and I was wondering if I could get some help. I have a list > of > IndexSearchers which are used in a MultiSearcher class. I use the > indexsearchers to get each indexreader and put them into a > MultiIndexReader. > > IndexReader[] readers = new IndexReader[searchables.length]; > > for (int i =0 ; i < searchables.length;i++) { > > IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; > > readers[i] = indexSearcher.getIndexReader(); > > IndexReader newReader = readers[i].reopen(); > > if (newReader != readers[i]) { > > readers[i].close(); > > } > > readers[i] = newReader; > > > > } > > multiReader = new MultiReader(readers); > > OpenBitSetFacetHitCounter facetHitCounter = > newOpenBitSetFacetHitCounter(); > > IndexSearcher indexSearcher = new IndexSearcher(multiReader); > > > I then use the indexseacher to do the facet stuff. I end the code with > closing the multireader. This is causing problems in another method > where I > do some other search as the indexreaders are closed. Is it ok to not > close > the multiindexreader or should I do some additional checks in the other > method to see if the indexreader is closed? > > > > Cheers > > > P.S. Hope that made sense...! > > > On Mon, Feb 23, 2009 at 7:20 AM, Amin Mohammed-Coleman < > ami...@gmail.com > >> wrote: >> >
Re: Faceted Search using Lucene
Amin Mohammed-Coleman wrote: Hi Thanks for your input. I would like to have a go at doing this myself first, Solr may be an option. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. -- I can moved the code out so that it is only created once and reused. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. -- This was something I thought about. I could move it out so that it's created once. However I presume inside my code i need to check whether the indexreaders are update to date. This needs to be synchronized as well I guess(?) Yes you should synchronize the check for whether the IndexReader is current. * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. -- So i need to extract the logic for reopening and provide a synchronisation mechanism. Yes. Ok. So I have some work to do. I'll refactor the code and see if I can get inline to your recommendations. On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandless < luc...@mikemccandless.com> wrote: On a quick look, I think there are a few problems with the code: * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. * You are over-incRef'ing (the reader.incRef inside the loop) -- I don't see a corresponding decRef. * You reopen and warm your searchers "live" (vs with BG thread); meaning the unlucky search request that hits a reopen pays the cost. This might be OK if the index is small enough that reopening & warming takes very little time. But if index gets large, making a random search pay that warming cost is not nice to the end user. It erodes their trust in you. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. You should consider simply using Solr -- it handles all this logic for you and has been well debugged with time... Mike Amin Mohammed-Coleman wrote: The reason for the indexreader.reopen is because I have a webapp which enables users to upload files and then search for the documents. If I don't reopen i'm concerned that the facet hit counter won't be updated. On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman wrote: Hi I have been able to get the code working for my scenario, however I have a question and I was wondering if I could get some help. I have a list of IndexSearchers which are used in a MultiSearcher class. I use the indexsearchers to get each indexreader and put them into a MultiIndexReader. IndexReader[] readers = new IndexReader[searchables.length]; for (int i =0 ; i < searchables.length;i++) { IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; readers[i] = indexSearcher.getIndexReader(); IndexReader newReader = readers[i].reopen(); if (newReader != readers[i]) { readers[i].close(); } readers[i] = newReader; } multiReader = new MultiReader(readers); OpenBitSetFacetHitCounter facetHitCounter = newOpenBitSetFacetHitCounter(); IndexSearcher indexSearcher = new IndexSearcher(multiReader); I then use the indexseacher to do the facet stuff. I end the code with closing the multireader. This is causing problems in another method where I do some other search as the indexreaders are closed. Is it ok to not close the multiindexreader or should I do some additional checks in the other method to see if the indexreader is closed? Cheers P.S. Hope that made sense...! On Mon, Feb 23, 2009 at 7:20 AM, Amin Mohammed-Coleman wrote: Hi Thanks just what I needed! Cheers Amin On 22 Feb 2009, at 16:11, Marcelo Ochoa wrote: Hi Amin: Please take a look a this blog post: http://sujitpal.blogspot.com/2007/04/lucene-search-within-search-with.html Best regards, Marcelo. On Sun, Feb 22, 2009 at 1:18 PM, Amin Mohammed-Coleman < ami...@gmail.com> wrote: Hi Sorry to re send this email but I was wondering if I could get some advice on this. Cheers Amin On 16 Feb 2009, at 20:37, Amin Mohammed-Coleman > wrote: Hi I am looking at building a faceted search using Lucene. I know that Solr comes with this built in, however I would like to try this by myself (something to add to my CV!). I have been looking around and I found that you can use the IndexReader and use TermVectors. This looks ok but I'm not sure how to filter the results so that a
Re: Faceted Search using Lucene
Hi Thanks for your input. I would like to have a go at doing this myself first, Solr may be an option. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. -- I can moved the code out so that it is only created once and reused. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. -- This was something I thought about. I could move it out so that it's created once. However I presume inside my code i need to check whether the indexreaders are update to date. This needs to be synchronized as well I guess(?) * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. -- So i need to extract the logic for reopening and provide a synchronisation mechanism. Ok. So I have some work to do. I'll refactor the code and see if I can get inline to your recommendations. On Sun, Mar 1, 2009 at 12:11 PM, Michael McCandless < luc...@mikemccandless.com> wrote: > > On a quick look, I think there are a few problems with the code: > > * I don't see any synchronization -- it looks like two search >requests are allowed into this method at the same time? Which is >dangerous... eg both (or, more) will wastefully reopen the >readers. > > * You are over-incRef'ing (the reader.incRef inside the loop) -- I >don't see a corresponding decRef. > > * You reopen and warm your searchers "live" (vs with BG thread); >meaning the unlucky search request that hits a reopen pays the >cost. This might be OK if the index is small enough that >reopening & warming takes very little time. But if index gets >large, making a random search pay that warming cost is not nice to >the end user. It erodes their trust in you. > > * You always make a new IndexSearcher and a new MultiSearcher even >when nothing has changed. This just generates unnecessary garbage >which GC then must sweep up. > > * You are creating a new Analyzer & QueryParser every time, also >creating unnecessary garbage; instead, they should be created once >& reused. > > You should consider simply using Solr -- it handles all this logic for > you and has been well debugged with time... > > Mike > > Amin Mohammed-Coleman wrote: > > The reason for the indexreader.reopen is because I have a webapp which >> enables users to upload files and then search for the documents. If I >> don't >> reopen i'm concerned that the facet hit counter won't be updated. >> >> On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman > >wrote: >> >> Hi >>> I have been able to get the code working for my scenario, however I have >>> a >>> question and I was wondering if I could get some help. I have a list of >>> IndexSearchers which are used in a MultiSearcher class. I use the >>> indexsearchers to get each indexreader and put them into a >>> MultiIndexReader. >>> >>> IndexReader[] readers = new IndexReader[searchables.length]; >>> >>> for (int i =0 ; i < searchables.length;i++) { >>> >>> IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; >>> >>> readers[i] = indexSearcher.getIndexReader(); >>> >>> IndexReader newReader = readers[i].reopen(); >>> >>> if (newReader != readers[i]) { >>> >>> readers[i].close(); >>> >>> } >>> >>> readers[i] = newReader; >>> >>> >>> >>> } >>> >>> multiReader = new MultiReader(readers); >>> >>> OpenBitSetFacetHitCounter facetHitCounter = >>> newOpenBitSetFacetHitCounter(); >>> >>> IndexSearcher indexSearcher = new IndexSearcher(multiReader); >>> >>> >>> I then use the indexseacher to do the facet stuff. I end the code with >>> closing the multireader. This is causing problems in another method >>> where I >>> do some other search as the indexreaders are closed. Is it ok to not >>> close >>> the multiindexreader or should I do some additional checks in the other >>> method to see if the indexreader is closed? >>> >>> >>> >>> Cheers >>> >>> >>> P.S. Hope that made sense...! >>> >>> >>> On Mon, Feb 23, 2009 at 7:20 AM, Amin Mohammed-Coleman >> >wrote: >>> >>> Hi Thanks just what I needed! Cheers Amin On 22 Feb 2009, at 16:11, Marcelo Ochoa wrote: Hi Amin: > Please take a look a this blog post: > > > http://sujitpal.blogspot.com/2007/04/lucene-search-within-search-with.html > Best regards, Marcelo. > > On Sun, Feb 22, 2009 at 1:18 PM, Amin Mohammed-Coleman < > ami...@gmail.com> > wrote: > > Hi >> >> Sorry to re send this email but I was wondering if I could get some >> advice >> on this. >> >> Cheers >> >> Amin >> >> On 16 Feb 2009, at 20:37, Amin Mohammed-Coleman >> wrote: >> >> Hi >
Re: Faceted Search using Lucene
On a quick look, I think there are a few problems with the code: * I don't see any synchronization -- it looks like two search requests are allowed into this method at the same time? Which is dangerous... eg both (or, more) will wastefully reopen the readers. * You are over-incRef'ing (the reader.incRef inside the loop) -- I don't see a corresponding decRef. * You reopen and warm your searchers "live" (vs with BG thread); meaning the unlucky search request that hits a reopen pays the cost. This might be OK if the index is small enough that reopening & warming takes very little time. But if index gets large, making a random search pay that warming cost is not nice to the end user. It erodes their trust in you. * You always make a new IndexSearcher and a new MultiSearcher even when nothing has changed. This just generates unnecessary garbage which GC then must sweep up. * You are creating a new Analyzer & QueryParser every time, also creating unnecessary garbage; instead, they should be created once & reused. You should consider simply using Solr -- it handles all this logic for you and has been well debugged with time... Mike Amin Mohammed-Coleman wrote: The reason for the indexreader.reopen is because I have a webapp which enables users to upload files and then search for the documents. If I don't reopen i'm concerned that the facet hit counter won't be updated. On Tue, Feb 24, 2009 at 8:32 PM, Amin Mohammed-Coleman >wrote: Hi I have been able to get the code working for my scenario, however I have a question and I was wondering if I could get some help. I have a list of IndexSearchers which are used in a MultiSearcher class. I use the indexsearchers to get each indexreader and put them into a MultiIndexReader. IndexReader[] readers = new IndexReader[searchables.length]; for (int i =0 ; i < searchables.length;i++) { IndexSearcher indexSearcher = (IndexSearcher)searchables[i]; readers[i] = indexSearcher.getIndexReader(); IndexReader newReader = readers[i].reopen(); if (newReader != readers[i]) { readers[i].close(); } readers[i] = newReader; } multiReader = new MultiReader(readers); OpenBitSetFacetHitCounter facetHitCounter = newOpenBitSetFacetHitCounter(); IndexSearcher indexSearcher = new IndexSearcher(multiReader); I then use the indexseacher to do the facet stuff. I end the code with closing the multireader. This is causing problems in another method where I do some other search as the indexreaders are closed. Is it ok to not close the multiindexreader or should I do some additional checks in the other method to see if the indexreader is closed? Cheers P.S. Hope that made sense...! On Mon, Feb 23, 2009 at 7:20 AM, Amin Mohammed-Coleman >wrote: Hi Thanks just what I needed! Cheers Amin On 22 Feb 2009, at 16:11, Marcelo Ochoa wrote: Hi Amin: Please take a look a this blog post: http://sujitpal.blogspot.com/2007/04/lucene-search-within-search-with.html Best regards, Marcelo. On Sun, Feb 22, 2009 at 1:18 PM, Amin Mohammed-Coleman > wrote: Hi Sorry to re send this email but I was wondering if I could get some advice on this. Cheers Amin On 16 Feb 2009, at 20:37, Amin Mohammed-Coleman wrote: Hi I am looking at building a faceted search using Lucene. I know that Solr comes with this built in, however I would like to try this by myself (something to add to my CV!). I have been looking around and I found that you can use the IndexReader and use TermVectors. This looks ok but I'm not sure how to filter the results so that a particular user can only see a subset of results. The next option I was looking at was something like Term term1 = new Term("brand", "ford"); Term term2 = new Term("brand", "vw"); Term[] termsArray = new Term[] { term1, term2 };un int[] docFreqs = indexSearcher.docFreqs(termsArray); The only problem here is that I have to provide the brand type each time a new brand is created. Again I'm not sure how I can filter the results here. It may be that I'm using the wrong api methods to do this. I would be grateful if I could get some advice on this. Cheers Amin P.S. I am basically trying to do something that displays the following Personal Contact (23) Business Contact (45) and so on.. -- Marcelo F. Ochoa http://marceloochoa.blogspot.com/ http://marcelo.ochoa.googlepages.com/home __ Want to integrate Lucene and Oracle? http://marceloochoa.blogspot.com/2007/09/running-lucene-inside-your-oracle-jvm.html Is Oracle 11g REST ready? http://marceloochoa.blogspot.com/2008/02/is-oracle-11g-rest-ready.html - To unsubscribe, e-mail: java-user-unsubscr...@lucene.apache.org For additional commands, e-mail: java-user-h...@lucene.apache.org - To u
N-grams with numbers and Shinglefilters
Hi, I'm trying to index (& search later) documents that contain tri-grams however they have the following form: <2 digit> <2 digit> Does the ShingleFilter work with numbers in the match ? Another complication, in future features I'd like to add optional digits like [<1 digit>] <2 digit> <2 digit> I suppose the ShingleFilter won't do it ? Any better advice ? Any help appreciated. -RB-