Github user kishorvpatil commented on a diff in the pull request:
https://github.com/apache/storm/pull/2920#discussion_r239231045
--- Diff:
storm-server/src/main/java/org/apache/storm/metricstore/NimbusMetricProcessor.java
---
@@ -24,7 +24,7 @@
public void processWorkerMetrics(Map<String, Object> conf,
WorkerMetrics metrics) throws MetricException {
try (NimbusClient client = NimbusClient.getConfiguredClient(conf))
{
client.getClient().processWorkerMetrics(metrics);
- } catch (TException e) {
--- End diff --
Could we instead of handling all exceptions, be more specific on capturing
`NimbusLeaderNotFoundException` ?
`catch (TException | NimbusLeaderNotFoundException e)`
---
