cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Jeff Jirsa (JIRA)" <>
Subject [jira] [Commented] (CASSANDRA-13396) Cassandra 3.10: ClassCastException in ThreadAwareSecurityManager
Date Mon, 03 Apr 2017 16:16:41 GMT


Jeff Jirsa commented on CASSANDRA-13396:

There are really three issues:

1) The existing comments were clearly inadequate, and that's been ninja'd into place. +1 on

2) Throwing a ClassCastException is objectively wrong. The patch fixes that, and should be

3) As a side effect, the patch allows other loggers, almost all of which are untested. The
assertion from [~snazy] is that doing so is dangerous, specifically citing past bugs where
other loggers which may do IO and cause sandbox access problems. That's a valid concern, and
worth a logged warning in my opinion.

Like [] (and I think [~blerer]) suggest above, I think Ed's patch+warning
makes sense to me.

If someone wants to "officially" support another logger in order to remove the warning, then
I think the burden is on them to open a proper ticket and demonstrate that it's sufficiently

> Cassandra 3.10: ClassCastException in ThreadAwareSecurityManager
> ----------------------------------------------------------------
>                 Key: CASSANDRA-13396
>                 URL:
>             Project: Cassandra
>          Issue Type: Bug
>            Reporter: Edward Capriolo
>            Priority: Minor

This message was sent by Atlassian JIRA

View raw message