[ https://issues.apache.org/jira/browse/SOLR-9360?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15634635#comment-15634635 ]
Erick Erickson commented on SOLR-9360: -------------------------------------- OK, the grep -w option does match whole words only, so this works. -D is a flag for grep (how universal I'm not sure) and at least one other place greps (incorrectly) just for jetty.port rather than jetty\.port so I'll just use the jetty\.port=$WOLR_PORT version There's a couple of greps for "solr.solr.home" as well that I'll fix on the way by. I took a brief look at the windows script and it uses another mechanism, so I'm not going to touch that one especially as I have no way to test it. > Solr script not properly checking SOLR_PID > ------------------------------------------ > > Key: SOLR-9360 > URL: https://issues.apache.org/jira/browse/SOLR-9360 > Project: Solr > Issue Type: Bug > Security Level: Public(Default Security Level. Issues are Public) > Components: scripts and tools > Affects Versions: 6.1 > Reporter: Alessandro Benedetti > Assignee: Erick Erickson > Attachments: SOLR_9360.patch > > > In the solr script we see in 3-4 areas this check : > SOLR_PID=`ps auxww | grep start\.jar | grep -w $SOLR_PORT | grep -v grep | > awk '{print $2}' | sort -r` > This can potentially prevent a solr instance to start in the case another > process by any chance contains the port int the command itself ( not > necessarily actually using the port) . > e.g. > java -server -Djetty.port=10504 -DSTOP.PORT=9504 -DSTOP.KEY=solrrocks > -DMASTER_CORE_URL=external-server:10500/solr -jar start.jar --module=http > A solr is running on 10504. > A new Solr will not be able to start on 10500 ( but actually the port is > free). > This should be replaced by a real check if the port is used ( like netstat or > similar) . -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org