[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-26 Thread Jonathan Ellis (JIRA)

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

Jonathan Ellis commented on CASSANDRA-6907:
---

committed

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7, 2.1 beta2
>
> Attachments: CASSANDRA-6907_v1.patch, CASSANDRA-6907_v2.patch, 
> CASSANDRA-6907_v3.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-24 Thread Joshua McKenzie (JIRA)

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

Joshua McKenzie commented on CASSANDRA-6907:


new patch attached.

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
> Attachments: CASSANDRA-6907_v1.patch, CASSANDRA-6907_v2.patch, 
> CASSANDRA-6907_v3.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-24 Thread Jonathan Ellis (JIRA)

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

Jonathan Ellis commented on CASSANDRA-6907:
---

Let's go ahead and do that, we can just leave it out of 3.0 when we merge so 
there's no need to worry about it outliving its usefulness.

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
> Attachments: CASSANDRA-6907_v1.patch, CASSANDRA-6907_v2.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-24 Thread Joshua McKenzie (JIRA)

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

Joshua McKenzie commented on CASSANDRA-6907:


Would be more robust, yes, but also represent more invasive changes for 
temporary OS-specific workarounds.  If we want to err on the side of robustness 
that should be a trivial change.

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
> Attachments: CASSANDRA-6907_v1.patch, CASSANDRA-6907_v2.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-24 Thread Jonathan Ellis (JIRA)

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

Jonathan Ellis commented on CASSANDRA-6907:
---

Wouldn't addressing in code be more robust for invocations via jmx as well as 
.bat?

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
> Attachments: CASSANDRA-6907_v1.patch, CASSANDRA-6907_v2.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-24 Thread Joshua McKenzie (JIRA)

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

Joshua McKenzie commented on CASSANDRA-6907:


Went with protection in the startup .bat file to force addition of -par on 
repair if it's not there and log to stdout.  Lighter touch and isolates changes 
to windows environment only.

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
> Attachments: CASSANDRA-6907_v1.patch
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)


[jira] [Commented] (CASSANDRA-6907) ignore snapshot repair flag on Windows

2014-03-21 Thread Jonathan Ellis (JIRA)

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

Jonathan Ellis commented on CASSANDRA-6907:
---

(I would prefer to leave the default of snapshot the same on both windows and 
linux, which means we should log a warning and ignore the flag instead of 
failing the operation.)

> ignore snapshot repair flag on Windows
> --
>
> Key: CASSANDRA-6907
> URL: https://issues.apache.org/jira/browse/CASSANDRA-6907
> Project: Cassandra
>  Issue Type: Bug
>  Components: Tools
>Reporter: Jonathan Ellis
>Assignee: Joshua McKenzie
> Fix For: 2.0.7
>
>
> Per discussion in CASSANDRA-4050, we should ignore the snapshot repair flag 
> on windows, and log a warning while proceeding to do non-snapshot repair.



--
This message was sent by Atlassian JIRA
(v6.2#6252)