/**
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you 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 org.apache.pulsar.client.impl;

import java.net.InetSocketAddress;
import java.util.List;
import java.util.Optional;
import java.util.concurrent.CompletableFuture;

import org.apache.commons.lang3.tuple.Pair;
import org.apache.pulsar.client.api.PulsarClientException;
import org.apache.pulsar.common.api.proto.PulsarApi.CommandGetTopicsOfNamespace.Mode;
import org.apache.pulsar.common.naming.NamespaceName;
import org.apache.pulsar.common.naming.TopicName;
import org.apache.pulsar.common.partition.PartitionedTopicMetadata;
import org.apache.pulsar.common.schema.SchemaInfo;

/**
 * Provides lookup service to find broker which serves given topic. It helps to
 * lookup
 * <ul>
 * <li><b>topic-lookup:</b> lookup to find broker-address which serves given
 * topic</li>
 * <li><b>Partitioned-topic-Metadata-lookup:</b> lookup to find
 * PartitionedMetadata for a given topic</li>
 * </ul>
 *
 */
public interface LookupService extends AutoCloseable {

    /**
     * Instruct the LookupService to switch to a new service URL for all subsequent requests
     */
    void updateServiceUrl(String serviceUrl) throws PulsarClientException;

    /**
     * Calls broker lookup-api to get broker {@link InetSocketAddress} which serves namespace bundle that contains given
     * topic.
     *
     * @param topicName
     *            topic-name
     * @return a pair of addresses, representing the logical and physical address of the broker that serves given topic
     */
    public CompletableFuture<Pair<InetSocketAddress, InetSocketAddress>> getBroker(TopicName topicName);

	/**
	 * Returns {@link PartitionedTopicMetadata} for a given topic.
	 *
	 * @param topicName topic-name
	 * @return
	 */
	public CompletableFuture<PartitionedTopicMetadata> getPartitionedTopicMetadata(TopicName topicName);

	/**
	 * Returns current SchemaInfo {@link SchemaInfo} for a given topic.
	 *
	 * @param topicName topic-name
	 * @return SchemaInfo
	 */
	public CompletableFuture<Optional<SchemaInfo>> getSchema(TopicName topicName);

	/**
	 * Returns specific version SchemaInfo {@link SchemaInfo} for a given topic.
	 *
	 * @param topicName topic-name
	 * @param version schema info version
	 * @return SchemaInfo
	 */
	public CompletableFuture<Optional<SchemaInfo>> getSchema(TopicName topicName, byte[] version);

	/**
	 * Returns broker-service lookup api url.
	 *
	 * @return
	 */
	public String getServiceUrl();

	/**
	 * Returns all the topics name for a given namespace.
	 *
	 * @param namespace : namespace-name
	 * @return
	 */
	public CompletableFuture<List<String>> getTopicsUnderNamespace(NamespaceName namespace, Mode mode);

}

@@ -2283,23 +2283,16 @@
  */%0A    
-public 
 Completa
@@ -2369,32 +2369,32 @@
 ame topicName);%0A
+
 %0A%09/**%0A%09 * Return
@@ -2493,39 +2493,32 @@
 * @return%0A%09 */%0A%09
-public 
 CompletableFutur
@@ -2727,39 +2727,32 @@
 chemaInfo%0A%09 */%0A%09
-public 
 CompletableFutur
@@ -2991,31 +2991,24 @@
 aInfo%0A%09 */%0A%09
-public 
 CompletableF
@@ -3154,23 +3154,16 @@
 n%0A%09 */%0A%09
-public 
 String g
@@ -3302,16 +3302,9 @@
  */%0A
+
 %09
-public 
 Comp

Remove redundant 'public' modifier in LookupService. (#6636)