You are viewing a plain text version of this content. The canonical link for it is here.
Posted to pr@cassandra.apache.org by "tommystendahl (via GitHub)" <gi...@apache.org> on 2025/02/24 12:45:39 UTC

[PR] CASSANDRA-20363 Add option to set custom FSErrorHandler [cassandra]

tommystendahl opened a new pull request, #3928:
URL: https://github.com/apache/cassandra/pull/3928

   patch by Tommy Stendahl; reviewed by TBD for CASSANDRA-20363


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org


Re: [PR] CASSANDRA-20363 Add option to set custom FSErrorHandler [cassandra]

Posted by "smiklosovic (via GitHub)" <gi...@apache.org>.
smiklosovic closed pull request #3928: CASSANDRA-20363 Add option to set custom FSErrorHandler
URL: https://github.com/apache/cassandra/pull/3928


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org


Re: [PR] CASSANDRA-20363 Add option to set custom FSErrorHandler [cassandra]

Posted by "smiklosovic (via GitHub)" <gi...@apache.org>.
smiklosovic commented on code in PR #3928:
URL: https://github.com/apache/cassandra/pull/3928#discussion_r1969993303


##########
src/java/org/apache/cassandra/service/CassandraDaemon.java:
##########
@@ -97,6 +98,7 @@
 import static java.util.concurrent.TimeUnit.NANOSECONDS;
 import static org.apache.cassandra.config.CassandraRelevantProperties.CASSANDRA_FOREGROUND;
 import static org.apache.cassandra.config.CassandraRelevantProperties.CASSANDRA_PID_FILE;
+import static org.apache.cassandra.config.CassandraRelevantProperties.CASSANDRA_FSERRORHANDLER;

Review Comment:
   @tommystendahl would you mind to rename this to `CASSANDRA_FS_ERROR_HANDLER` ? 



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@cassandra.apache.org
For additional commands, e-mail: pr-help@cassandra.apache.org