Random test failure

2009-05-16 Thread Earwin Burrfoot
Running latest lucene trunk with some patches applied, but they do not touch IndexWriter and friends anywhere. Happened once, I failed to reproduce it, with and without patches. Java(TM) SE Runtime Environment (build 1.6.0_07-b06-153) Java HotSpot(TM) 64-Bit Server VM (build 1.6.0_07-b06-57,

Re: svn commit: r774718 [2/3] - in /lucene/java/trunk: ./ contrib/analyzers/src/java/org/apache/lucene/analysis/ar/ contrib/analyzers/src/java/org/apache/lucene/analysis/cn/ contrib/analyzers/src/j

2009-05-16 Thread Michael McCandless
On Fri, May 15, 2009 at 11:38 PM, Otis Gospodnetic otis_gospodne...@yahoo.com wrote: Is that Copyright 2009 www.imdict.net allowed to be in there? Hmm --- good question. According to this: http://www.apache.org/legal/src-headers.html I think it is not. Ie it should be moved to NOTICE.txt.

RE: Random test failure

2009-05-16 Thread Uwe Schindler
Hi Earwin, As this error occurs directly inside the HashMap and so is Java's code, I think this is a) an error in Java's HashMap implementation (haha, unlikely). But I tend to say, Java's HashMap is correct... b) missing synchronization around the HashMap, so it is modified at the same time. So

RE: svn commit: r774718 [2/3] - in /lucene/java/trunk: ./ contrib/analyzers/src/java/org/apache/lucene/analysis/ar/ contrib/analyzers/src/java/org/apache/lucene/analysis/cn/ contrib/analyzers/src/java

2009-05-16 Thread Uwe Schindler
Before committing I have seen this, too, but then forgot about it (any many other things I added to the case after the commit)! - Uwe Schindler H.-H.-Meier-Allee 63, D-28213 Bremen http://www.thetaphi.de eMail: u...@thetaphi.de -Original Message- From: Michael McCandless

Re: svn commit: r774718 [2/3] - in /lucene/java/trunk: ./ contrib/analyzers/src/java/org/apache/lucene/analysis/ar/ contrib/analyzers/src/java/org/apache/lucene/analysis/cn/ contrib/analyzers/src/j

2009-05-16 Thread Michael McCandless
OK I committed the fix. Thanks Otis! Mike On Sat, May 16, 2009 at 5:37 AM, Uwe Schindler u...@thetaphi.de wrote: Before committing I have seen this, too, but then forgot about it (any many other things I added to the case after the commit)! - Uwe Schindler H.-H.-Meier-Allee 63,

Re: Random test failure

2009-05-16 Thread Michael McCandless
MockRAMDirectory already tries to synchronize access to its openFiles HashSet... but it must be missing somewhere. Mike 2009/5/16 Uwe Schindler u...@thetaphi.de: Hi Earwin, As this error occurs directly inside the HashMap and so is Java's code, I think this is a) an error in Java's HashMap

Re: Random test failure

2009-05-16 Thread Michael McCandless
Oh, it's actually MockRAMDirectory.createdFiles that hit this, and I see that we're inconsistent on how we synchronize access to it. I'll fix. Mike On Sat, May 16, 2009 at 5:10 AM, Earwin Burrfoot ear...@gmail.com wrote: Running latest lucene trunk with some patches applied, but they do not

Re: svn commit: r774718 [2/3] - in /lucene/java/trunk: ./ contrib/analyzers/src/java/org/apache/lucene/analysis/ar/ contrib/analyzers/src/java/org/apache/lucene/analysis/cn/ contrib/analyzers/src/j

2009-05-16 Thread Gao Pinker
According to the license page http://apache.org/licenses/LICENSE-2.0.html#apply, especially APPENDIX: How to apply the Apache License to your work Copyright [] [name of copyright owner] So I included the notice. But as I saw most of the contributors don't include this, you can remove it. By

Re: svn commit: r774718 [2/3] - in /lucene/java/trunk: ./ contrib/analyzers/src/java/org/apache/lucene/analysis/ar/ contrib/analyzers/src/java/org/apache/lucene/analysis/cn/ contrib/analyzers/src/j

2009-05-16 Thread Michael McCandless
OK thanks Gao. I removed those two files, and fixed the copyrights. Mike On Sat, May 16, 2009 at 9:34 AM, Gao Pinker xiaoping...@gmail.com wrote: According to the license page http://apache.org/licenses/LICENSE-2.0.html#apply, especially APPENDIX: How to apply the Apache License to your

[jira] Commented: (LUCENE-1629) contrib intelligent Analyzer for Chinese

2009-05-16 Thread Koji Sekiguchi (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12710118#action_12710118 ] Koji Sekiguchi commented on LUCENE-1629: bq. koji, have you considered using icu

[jira] Updated: (LUCENE-1640) MockRAMDirectory (used only by unit tests) has some synchronization problems

2009-05-16 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1640?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael McCandless updated LUCENE-1640: --- Attachment: LUCENE-1640.patch Attached patch. I switched to simply always

[jira] Created: (LUCENE-1640) MockRAMDirectory (used only by unit tests) has some synchronization problems

2009-05-16 Thread Michael McCandless (JIRA)
MockRAMDirectory (used only by unit tests) has some synchronization problems Key: LUCENE-1640 URL: https://issues.apache.org/jira/browse/LUCENE-1640 Project: Lucene - Java

[jira] Resolved: (LUCENE-1634) LogMergePolicy should use the number of deleted docs when deciding which segments to merge

2009-05-16 Thread Michael McCandless (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1634?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Michael McCandless resolved LUCENE-1634. Resolution: Fixed Thanks Yasuhiro! LogMergePolicy should use the number of

[jira] Commented: (LUCENE-1629) contrib intelligent Analyzer for Chinese

2009-05-16 Thread Mingfai Ma (JIRA)
[ https://issues.apache.org/jira/browse/LUCENE-1629?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=12710147#action_12710147 ] Mingfai Ma commented on LUCENE-1629: i'm not sure if the character mapping is a