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

Hoss Man commented on SOLR-8689:
--------------------------------

My thoughts in no particular order -- w/o having looked at /understood the 
patch (i'm not familar with most windows batch file syntax)...

bq. BTW, I set this as blocker, as we cannot release Solr 7 at the same time 
like Java 9 and it won't work.

I don't think "solr + windows + jdk9" is an important enough use case to 
warrant holding up lucene/solr 7.  We can release note that solr.cmd startup 
script doesn't work in that permutation and focus on fixing later (if anyone is 
really hardcore about using solr + windows + jdk9 they can probably work around 
using the solr.cmd script)

bq. The GC_LOG_OPTS used as it was before the change on Java <= 8, but for Java 
9 it is ignored. I have no better idea how to do this. IMHO, this is better 
than not working at all! I added documentation for this.

My personal opinion is that it would be better to continue to support things as 
best we can for existing solr users who may upgrade solr w/o caring about jdk9 
at all -- ie: keep supporting/using GC_LOG_OPTS -- but in the special in the 
case where they use jdk9 _and_ try to specify a value for the GC_LOG_OPTS ... 
we should hard fail w/message that solr can't support GC_LOG_OPTS  when using 
jdk9.  

I believe you said in your current patch you "ignored" GC_LOG_OPTS when using 
jdk9? ... i think that's a mistake, as existing solr users who may have already 
set GC_LOG_OPTS and then later upgrade to jdk9 will get confusing silently 
different behavior.  better to fail fast and be clear that they can't do that 
(unless/until we find a better solution)


> bin/solr.cmd does not start with recent Verona builds of Java 9 because of 
> version parsing issue
> ------------------------------------------------------------------------------------------------
>
>                 Key: SOLR-8689
>                 URL: https://issues.apache.org/jira/browse/SOLR-8689
>             Project: Solr
>          Issue Type: Bug
>          Components: scripts and tools
>    Affects Versions: 5.5, 6.0
>         Environment: Windows 7
>            Reporter: Uwe Schindler
>            Assignee: Uwe Schindler
>            Priority: Blocker
>              Labels: Java9
>             Fix For: 7.0, master (8.0), 7.1
>
>         Attachments: SOLR-8689.patch, SOLR-8689.patch, SOLR-8689.patch, 
> SOLR-8689.patch
>
>
> At least on Windows, Solr 5.5 does not start with the shell script using a 
> Verona-Java-9 JDK:
> {noformat}
> *****************************************************
> JAVA_HOME = C:\Program Files\Java\jdk-9
> java version "9-ea"
> Java(TM) SE Runtime Environment (build 
> 9-ea+105-2016-02-11-003336.javare.4433.nc)
> Java HotSpot(TM) 64-Bit Server VM (build 
> 9-ea+105-2016-02-11-003336.javare.4433.nc, mixed mode)
> *****************************************************
> C:\Users\Uwe Schindler\Desktop\solr-5.5.0\bin>solr start
> ERROR: Java 1.7 or later is required to run Solr. Current Java version is: 
> 9-ea
> {noformat}
> I don't know if this is better with Linux, but I assume the version parsing 
> is broken (e.g., String#startsWith, interpret as floating point number,...)
> We should fix this before Java 9 gets released! The version numbering scheme 
> changed completely: http://openjdk.java.net/jeps/223



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org
For additional commands, e-mail: dev-h...@lucene.apache.org

Reply via email to