[
https://issues.apache.org/jira/browse/KAFKA-2605?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Ismael Juma resolved KAFKA-2605.
--------------------------------
Resolution: Won't Fix
Scala is being replaced with Java, so this is unlikely to get done aside from
the places where it's already been done.
> Replace `catch: Throwable` clauses with `NonFatal` or `NonControl`
> ------------------------------------------------------------------
>
> Key: KAFKA-2605
> URL: https://issues.apache.org/jira/browse/KAFKA-2605
> Project: Kafka
> Issue Type: Improvement
> Affects Versions: 0.11.0.0, 1.0.0
> Reporter: Ismael Juma
> Assignee: Jin Xing
> Priority: Major
> Labels: newbie
>
> The Kafka codebase includes a number of instances where we do `catch t:
> Throwable` where we should really be doing `catch NonFatal(t)` or `catch
> NonControl(t)` where `NonFatal` is part of the standard library and
> `NonControl` is something like:
> {code}
> object NonControl {
> def apply(t: Throwable): Boolean = t match {
> case _: ControlThrowable => false
> case _ => true
> }
> def unapply(t: Throwable): Option[Throwable] = if (apply(t)) Some(t) else
> None
> }
> {code}
> We can also use `NonControl` to replace cases like (it's more concise and has
> the same behaviour):
> {code}
> case e: ControlThrowable => throw e
> case e: Throwable => ...
> {code}
--
This message was sent by Atlassian Jira
(v8.20.10#820010)
