[ 
https://issues.apache.org/jira/browse/LUCENE-1540?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12989508#comment-12989508
 ] 

Shai Erera commented on LUCENE-1540:
------------------------------------

We're very close indeed !

* Maybe instead of moving the unzip method to LuceneTestCase, you can put it as 
a static method in _TestUtil? LTC is crowded enough to be added more 
functionality :). Also, _TestUtil already has a rmDir method, I think we should 
use it? I would also do the same for fullTempDir.

* The method pathType(File f) in TrecDocParser -- maybe instead of walking up 
the path elements you can obtain its full absolute path (which is a String) and 
then do indexOf() checks for the 4 types? It will simplify matters IMO.

* stripTags:
** Typo in TDP: unmodofied --> unmodified.
** Maybe we can use String.replaceAll() which takes a regex? This is not 
critical ...
** Does stripTags strips off tags of the HTML content? Or is it used for the 
TREC types other than GOV2?

* In TrecContentSource, can you replace TrecParserByPath.pathType to 
TrecDocParser.pathType?
* Also, do we still need TrecParserByPath? I don't see that it's used.

> Improvements to contrib.benchmark for TREC collections
> ------------------------------------------------------
>
>                 Key: LUCENE-1540
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1540
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: contrib/benchmark
>            Reporter: Tim Armstrong
>            Assignee: Doron Cohen
>            Priority: Minor
>             Fix For: 3.1, 4.0
>
>         Attachments: LUCENE-1540.patch, LUCENE-1540.patch, LUCENE-1540.patch, 
> trecdocs.zip
>
>
> The benchmarking utilities for  TREC test collections (http://trec.nist.gov) 
> are quite limited and do not support some of the variations in format of 
> older TREC collections.  
> I have been doing some benchmarking work with Lucene and have had to modify 
> the package to support:
> * Older TREC document formats, which the current parser fails on due to 
> missing document headers.
> * Variations in query format - newlines after <title> tag causing the query 
> parser to get confused.
> * Ability to detect and read in uncompressed text collections
> * Storage of document numbers by default without storing full text.
> I can submit a patch if there is interest, although I will probably want to 
> write unit tests for the new functionality first.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to