[
https://issues.apache.org/jira/browse/LUCENE-2353?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Shai Erera updated LUCENE-2353:
-------------------------------
Attachment: LUCENE-2353.patch
The fix is only relevant to get(String, String) and not to all other
get(String, <type>) variants.
Benchmark test passed but after I svn up (to include the latest parallel test
thing) the test just sits idle (after finishing), waiting for something. If I
run the tests in eclipse they pass. So I'm guessing it's a problem w/ my env.
or build.xml?
I also tried 'ant clean test' from within benchmark, but it didn't help. I then
tried 'ant clean' from root, and 'ant test' from benchmark, but the test just
keeps waiting on WriteLineDocTaskTest, on this line:
[junit] ------------> config properties:
[junit] directory = RAMDirectory
[junit] doc.maker =
org.apache.lucene.benchmark.byTask.tasks.WriteLineDocTaskTest$JustDateDocMaker
[junit] line.file.out =
D:\dev\lucene\lucene-trunk\build\contrib\benchmark\test\W\one-line
[junit] -------------------------------
I think this can go in (if it passes on someone else's machine, while I figure
out what's wrong in my env. separately.
> Config incorrectly handles Windows absolute pathnames
> -----------------------------------------------------
>
> Key: LUCENE-2353
> URL: https://issues.apache.org/jira/browse/LUCENE-2353
> Project: Lucene - Java
> Issue Type: Bug
> Components: contrib/benchmark
> Reporter: Shai Erera
> Fix For: 3.1
>
> Attachments: LUCENE-2353.patch
>
>
> I have no idea how no one ran into this so far, but I tried to execute an
> .alg file which used ReutersContentSource and referenced both docs.dir and
> work.dir as Windows absolute pathnames (e.g. d:\something). Surprisingly, the
> run reported an error of missing content under benchmark\work\something.
> I've traced the problem back to Config, where get(String, String) includes
> the following code:
> {code}
> if (sval.indexOf(":") < 0) {
> return sval;
> }
> // first time this prop is extracted by round
> int k = sval.indexOf(":");
> String colName = sval.substring(0, k);
> sval = sval.substring(k + 1);
> ...
> {code}
> It detects ":" in the value and so it thinks it's a per-round property, thus
> stripping "d:" from the value ... fix is very simple:
> {code}
> if (sval.indexOf(":") < 0) {
> return sval;
> } else if (sval.indexOf(":\\") >= 0) {
> // this previously messed up absolute path names on Windows. Assuming
> // there is no real value that starts with \\
> return sval;
> }
> // first time this prop is extracted by round
> int k = sval.indexOf(":");
> String colName = sval.substring(0, k);
> sval = sval.substring(k + 1);
> {code}
> I'll post a patch w/ the above fix + test shortly.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
