[ 
https://issues.apache.org/jira/browse/SOLR-12759?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16618178#comment-16618178
 ] 

Hoss Man commented on SOLR-12759:
---------------------------------

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-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to