Skip to content

Commit 909b106

Browse files
committed
fix(kafka): remove panic on unknown KafkaError
1 parent 12fbe39 commit 909b106

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

bridge-core/src/kafka/mod.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,6 +27,8 @@ pub enum KafkaError {
2727
ClientCreationError(#[source] rdkafka::error::KafkaError),
2828
#[error("Failed to fetch metadata: {0}")]
2929
MetadataFetchFailed(#[source] rdkafka::error::KafkaError),
30+
#[error("Unknown Kafka error: {0}")]
31+
Unknown(String),
3032
}
3133

3234
impl From<rdkafka::error::KafkaError> for KafkaError {
@@ -59,9 +61,7 @@ impl From<rdkafka::error::KafkaError> for KafkaError {
5961
RDKafkaError::StoreOffset(err) => KafkaError::Generic(format_rdkafka_code(err)),
6062
RDKafkaError::Subscription(ref err) => KafkaError::Generic(err.to_string()),
6163
RDKafkaError::Transaction(err) => KafkaError::Generic(err.to_string()),
62-
_ => {
63-
panic!("Unhandled KafkaError");
64-
}
64+
other => KafkaError::Unknown(format!("{:?}", other)),
6565
}
6666
}
6767
}

0 commit comments

Comments
 (0)