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

Cyril Scetbon commented on CASSANDRA-8594:
------------------------------------------

bin/cassandra only sets the dedicated cassandra parameter 
[here|https://github.com/cscetbon/cassandra/blob/cassandra-2.0.10/bin/cassandra#L139]
 which doesn't write the pid at all. In 1.2, jsvc was used and it was this tool 
that [was setting the 
pid|https://github.com/apache/cassandra/blob/cassandra-1.2.13/debian/init#L136-L139].
 But now that we don't use it (2.0+)  we use 
[start-stop-daemon|https://github.com/cscetbon/cassandra/blob/cassandra-2.0.10/debian/init#L96]
 without asking it to create the pidfile. And as bin/cassandra says that it 
[stores the 
pid|https://github.com/cscetbon/cassandra/blob/cassandra-2.0.10/bin/cassandra#L21]
 if we use -p pidfile, I think it's time to implement it in cassandra. 

> pidfile is never filled by cassandra
> ------------------------------------
>
>                 Key: CASSANDRA-8594
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-8594
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Cyril Scetbon
>            Assignee: Cyril Scetbon
>             Fix For: 2.0.13
>
>
> The pid file is never filled by cassandra. there is only a File object that 
> is created with [those 
> lines|https://github.com/cscetbon/cassandra/blob/cassandra-2.0.10/src/java/org/apache/cassandra/service/CassandraDaemon.java#L498-L501]
> Here is a 
> [fix|https://github.com/cscetbon/cassandra/commit/d0c5e0c9be00e48e6d0cd0de208c53274f1919c0.patch]
>  that writes the current PID into the pidfile



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to