From 4866563de262eaae5ae8646cda207dcdece21272 Mon Sep 17 00:00:00 2001
From: Rajan <dhabalia.me@gmail.com>
Date: Thu, 6 Apr 2017 15:50:21 -0700
Subject: [PATCH] return if serviceNotReady while creating topic (#336)

---
 .../pulsar/broker/service/BrokerService.java      |  1 +
 .../utils/FixedColumnLengthTableMaker.java        | 15 +++++++++++++++
 2 files changed, 16 insertions(+)

diff --git a/pulsar-broker/src/main/java/com/yahoo/pulsar/broker/service/BrokerService.java b/pulsar-broker/src/main/java/com/yahoo/pulsar/broker/service/BrokerService.java
index 3f965e423bb..12983e16b5f 100644
--- a/pulsar-broker/src/main/java/com/yahoo/pulsar/broker/service/BrokerService.java
+++ b/pulsar-broker/src/main/java/com/yahoo/pulsar/broker/service/BrokerService.java
@@ -472,6 +472,7 @@ public class BrokerService implements Closeable, ZooKeeperCacheListener<Policies
             String msg = String.format("Namespace is being unloaded, cannot add topic %s", topic);
             log.warn(msg);
             topicFuture.completeExceptionally(new ServiceUnitNotReadyException(msg));
+            return;
         }
 
         getManagedLedgerConfig(destinationName).thenAccept(config -> {
diff --git a/pulsar-testclient/src/main/java/com/yahoo/pulsar/testclient/utils/FixedColumnLengthTableMaker.java b/pulsar-testclient/src/main/java/com/yahoo/pulsar/testclient/utils/FixedColumnLengthTableMaker.java
index fa7016eac0c..3391626cd35 100644
--- a/pulsar-testclient/src/main/java/com/yahoo/pulsar/testclient/utils/FixedColumnLengthTableMaker.java
+++ b/pulsar-testclient/src/main/java/com/yahoo/pulsar/testclient/utils/FixedColumnLengthTableMaker.java
@@ -1,3 +1,18 @@
+/**
+ * Copyright 2016 Yahoo Inc.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
 package com.yahoo.pulsar.testclient.utils;
 
 import java.util.Objects;
-- 
GitLab