On Sat, 2006-05-13 at 01:38 +0200, karl wettin wrote:
> here are my comments.

It seems to run fine on Lists. But I have only been running it on an
issue 550 index, so I have no idea how the Directory implementation
index will take it.

No test case, no nothing. Just tried it in live environment for a couple
of minutes.

> 
> 
> > StandardTokenizer:
> >   private java.util.Vector jj_expentries
> 
> If the standard analyzer is not thread safe, does this make sense? Looks
> genereted in my eyes. Did not look at the code.
> 
> > 
> > Document:
> >   List fields = new Vector();
> 
> ArrayList. People don't add fields from multiple threads?
> 
> > IndexWriter:
> >   addIndexes(IndexReader[])
> >     Vector segmentsToDelete
> >   deleteFiles(Vector)
> >     Vector deletable
> >   deleteSegments(Vector)
> >     Vector deletable
> >   mergeSegments(int, int)
> >     Vector segmentsToDelete
> >   readDeleteableFiles()
> >     Vector results
> 
> Located in methods. Vector makes no sense at all?
> 
> > 
> > SegmentMerger:
> >   private Vector readers
> 
> The IndexWriter keeps this instance synchronized?
> 
> >  
> > TermVectorWriter:
> >   terms = new Vector();
> 
> The IndexWriter keeps this instance synchronized?
> 
> > 
> > MultiFieldQueryParser:
> >   get(AnyKindOf)Query
> >     Vector clauses
> 
> 
> In methods. Vector makes no sense? Query parser is not thread safe, does
> this have to be?
> 
> > 
> > QueryParser:
> >   private java.util.Vector jj_expentries
> >   Query(String)
> >     Vector caluses
> >   getFieldQuery(String, String)
> >     Vector v; // tokens from stream
> 
> 
> In methods. Vector makes no sense? Query parser is not thread safe, does
> this have to be?
>   
> > BooleanQuery:
> >   private Vector clauses
> 
> People don't build a query from multiple threads?
> 
> > 
> > BooleanQuery.BooleanWeight:
> >   private Vector weights
> 
> People don't build a query from multiple threads?
> 
> > Hits:
> >   private Vector hitDocs
> 
> Parallel hits collection might require this?
> 
> > MultiPhraseQuery:
> >   private Vector positions
> 
> Unsure.
> 
> > PhraseQuery:
> >   private Vector positions
> >   private Vector terms
> 
> People don't build a query from multiple threads?
> 
> 
> > RAMFile:
> >   private Vector buffers
> 
> The IndexWriter keeps this instance synchronized?
> 
> 
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [EMAIL PROTECTED]
> For additional commands, e-mail: [EMAIL PROTECTED]
> 
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to