rmuir commented on PR #903: URL: https://github.com/apache/lucene/pull/903#issuecomment-1130814798
See JIRA issue for more background and example data files: https://issues.apache.org/jira/browse/LUCENE-10579 When reviewing the code, it may not be obvious that currently we are parsing a very-noisy **28.4MB** of stdout today, with multiple processes and threads all printing to it. Then we are parsing it with regular expressions. It makes the parsing flaky. Rather than run the test with `-Dtests.verbose=true` and try to parse thru megabytes of this stuff, we can just list the .zip files that the test uses. We still list all `*.cfs` files basically, and let the smoketester deal with all the comparisons it currently does against the apache archive. This is basically the minimal fix, of course we could implement the test completely differently, but I kinda like its heroic efforts to cross-check apache archive releases against our backwards compatibility tests. I just don't want it flaky as smoketests take hours for me. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
