You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucene.apache.org by "Hoss Man (JIRA)" <ji...@apache.org> on 2018/09/17 21:35:00 UTC
[jira] [Comment Edited] (SOLR-12759) Disable
ExtractingRequestHandlerTest on JDK 11
[ https://issues.apache.org/jira/browse/SOLR-12759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16618178#comment-16618178 ]
Hoss Man edited comment on SOLR-12759 at 9/17/18 9:34 PM:
----------------------------------------------------------
nitpick...
in the case of "broken in current RC of third-party X" type test assumptions/workarounds, i think we should try as much as possible to ensure that the the assumption/workaround starts failing loudly once the underlying third-party bug is fixed, so we don't inadvertently leave a bunch of tests disabled forever.
Maybe something like this (unless i've got my logic backwards)...
{code:java}
final boolean brokenJDKDates = someMethodThatReturnsFalseIfJDK11IsBroken();
if (brokenJDKDates) {
asumeTrue("SOLR-12759 JDK 11 RCs currently don't work",
System.getProperty("java.version").startsWith("11"));
} else {
assertFalse("SOLR-12759 JDK 11 RCs are not expected to work, apparently they do now, test should be updated",
System.getProperty("java.version").startsWith("11"));
}{code}
was (Author: hossman):
nitpick...
in the case of "broken in current RC of third-party X" type test assumptions/workarounds, i think we should try as much as possible to ensure that the the assumption/workaround starts failing loudly once the underlying third-party bug is fixed, so we don't inadvertently leave a bunch of tests disabled forever.
Maybe something like this (unless i've got my logic backwards)...
{code:java}
final boolean brokenJDKDates = someMethodThatReturnsFalseIfJDK11IsBroken();
if (brokenJDKDates) {
asumeTrue("SOLR-12759 JDK 11 RCs currently don't work",
System.getProperty("java.version").startsWith("11"));
} else {
asumeFalse("SOLR-12759 JDK 11 RCs are not expected to work, apparently they do now, test should be updated",
System.getProperty("java.version").startsWith("11"));
}{code}
> Disable ExtractingRequestHandlerTest on JDK 11
> ----------------------------------------------
>
> Key: SOLR-12759
> URL: https://issues.apache.org/jira/browse/SOLR-12759
> Project: Solr
> Issue Type: Improvement
> Security Level: Public(Default Security Level. Issues are Public)
> Components: contrib - Solr Cell (Tika extraction)
> Environment: JDK 11 and Tika 1.x
> Reporter: David Smiley
> Assignee: David Smiley
> Priority: Minor
> Fix For: 7.6
>
>
> ExtractingRequestHandlerTest has failed on a JDK 11 RC due to two conspiring problems: (A) Tika 1.x sometimes calls Date.toString() when extracting metadata (unreleased 2.x will fix this), (B) JDK 11 RC has a bug in some locales like Arabic in which a Date.toString() will have a timezone offset using its locale's characters for the digits instead of using EN_US.
> I'll add an "assume" check so we don't see failures about this.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org