Re: svn commit: r168213 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/store/FSDirectory.java

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r168213 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/store/FSDirectory.java

Doug Cutting
I'd prefer if the list of file extensions was in a single place, and
that place should be somewhere in the index package, not in the store
package.

Doug

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

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r168213 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/store/FSDirectory.java

Daniel Naber
On Thursday 05 May 2005 01:40, Doug Cutting wrote:

> I'd prefer if the list of file extensions was in a single place, and
> that place should be somewhere in the index package, not in the store
> package.

I will try to move it. BTW, index/SegmentReader.java refers to an extension
"tvp" which I cannot find anywhere. I guess this reference can be removed?

Regards
 Daniel

--
http://www.danielnaber.de

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

Reply | Threaded
Open this post in threaded view
|

Re: svn commit: r168213 - in /lucene/java/trunk: CHANGES.txt src/java/org/apache/lucene/store/FSDirectory.java

Daniel Naber
In reply to this post by Doug Cutting
On Thursday 05 May 2005 01:40, Doug Cutting wrote:

> I'd prefer if the list of file extensions was in a single place, and
> that place should be somewhere in the index package, not in the store
> package.

I attached a new patch here:
http://issues.apache.org/bugzilla/show_bug.cgi?id=34695

Regards
 Daniel

--
http://www.danielnaber.de

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