[Fix][client]Fix status html endpoint

conf/standalone.conf

@@ -212,7 +212,7 @@ clientLibraryVersionCheckEnabled=false
 
 # Path for the file used to determine the rotation status for the broker when responding
 # to service discovery health checks
-statusFilePath=/usr/local/apache/htdocs
+statusFilePath=
 
 # Max number of unacknowledged messages allowed to receive messages by a consumer on a shared subscription. Broker will stop sending
 # messages to consumer once, this limit reaches until consumer starts acknowledging messages back

pulsar-broker/src/main/java/org/apache/pulsar/broker/web/WebService.java

@@ -360,12 +360,37 @@ public void addServlet(String path, ServletHolder servletHolder, boolean require
     public void addStaticResources(String basePath, String resourcePath) {
         ContextHandler capHandler = new ContextHandler();
         capHandler.setContextPath(basePath);
-        ResourceHandler resHandler = new ResourceHandler();
-        resHandler.setBaseResource(Resource.newClassPathResource(resourcePath));
-        resHandler.setEtags(true);
-        resHandler.setCacheControl(WebService.HANDLER_CACHE_CONTROL);
-        capHandler.setHandler(resHandler);
-        handlers.add(capHandler);
+        if (resourcePath != null && !resourcePath.isEmpty()) {
+            ResourceHandler resHandler = new ResourceHandler();
+            resHandler.setBaseResource(Resource.newClassPathResource(resourcePath));
+            resHandler.setEtags(true);
+            resHandler.setCacheControl(WebService.HANDLER_CACHE_CONTROL);
+            capHandler.setHandler(resHandler);
+            handlers.add(capHandler);
+         } else {
+            // If statusFilePath is not set, return OK when the broker is ready, or 404 if not.
+            capHandler.setHandler(new DefaultHandler() {
+                @Override
+                public void handle(String target, org.eclipse.jetty.server.Request baseRequest,
+                                   javax.servlet.http.HttpServletRequest request,
+                                   javax.servlet.http.HttpServletResponse response) throws IOException {
+                     if (target.equals("/status.html") ) {
+                        if(pulsar.getState() == PulsarService.State.Started) {
+                            response.setContentType("text/plain");
+                            response.setStatus(HttpServletResponse.SC_OK);
+                            response.getWriter().write("OK");
+                            baseRequest.setHandled(true);
+                         } else {
+                             response.setStatus(HttpServletResponse.SC_NOT_FOUND);
+                             baseRequest.setHandled(true);
+                         }
+                      } else {
+                         super.handle(target, baseRequest, request, response);
+                      }
+                }
+            });
+           handlers.add(capHandler);
+        }
     }
 
     public void start() throws PulsarServerException {

pulsar-client-api/src/main/java/org/apache/pulsar/client/api/Schema.java

@@ -284,8 +284,8 @@ default Optional<Object> getNativeSchema() {
      * @return a Schema instance
      */
     static <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> PROTOBUF(Class<T> clazz) {
-        return DefaultImplementation.getDefaultImplementation()
-                .newProtobufSchema(SchemaDefinition.builder().withPojo(clazz).build());
+        return DefaultImplementation.getDefaultImplementation().newProtobufSchema(clazz);
+
     }
 
     /**
@@ -297,16 +297,15 @@ static <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> PROTOBUF(Cla
     static <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> PROTOBUF(SchemaDefinition<T> schemaDefinition) {
         return DefaultImplementation.getDefaultImplementation().newProtobufSchema(schemaDefinition);
     }
-
     /**
      * Create a Protobuf-Native schema type by extracting the fields of the specified class.
      *
      * @param clazz the Protobuf generated class to be used to extract the schema
      * @return a Schema instance
      */
     static <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> PROTOBUF_NATIVE(Class<T> clazz) {
-        return DefaultImplementation.getDefaultImplementation()
-                .newProtobufNativeSchema(SchemaDefinition.builder().withPojo(clazz).build());
+        return DefaultImplementation.getDefaultImplementation().newProtobufNativeSchema(clazz);
+
     }
 
     /**
@@ -327,8 +326,7 @@ static <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> PROTOBUF_NAT
      * @return a Schema instance
      */
     static <T> Schema<T> AVRO(Class<T> pojo) {
-        return DefaultImplementation.getDefaultImplementation()
-                .newAvroSchema(SchemaDefinition.builder().withPojo(pojo).build());
+        return DefaultImplementation.getDefaultImplementation().newAvroSchema(pojo);
     }
 
     /**
@@ -348,8 +346,7 @@ static <T> Schema<T> AVRO(SchemaDefinition<T> schemaDefinition) {
      * @return a Schema instance
      */
     static <T> Schema<T> JSON(Class<T> pojo) {
-        return DefaultImplementation.getDefaultImplementation()
-                .newJSONSchema(SchemaDefinition.builder().withPojo(pojo).build());
+        return DefaultImplementation.getDefaultImplementation().newJSONSchema(pojo);
     }
 
     /**

pulsar-client-api/src/main/java/org/apache/pulsar/client/internal/PulsarClientImplementationBinding.java

@@ -122,6 +122,14 @@ <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufNativeSc
 
     <T> Schema<T> newJSONSchema(SchemaDefinition schemaDefinition);
 
+    <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufSchema(Class<T> clazz);
+
+    <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufNativeSchema(Class<T> clazz);
+
+    <T> Schema<T> newJSONSchema(Class<T> pojo);
+
+    <T> Schema<T> newAvroSchema(Class<T> pojo);
+
     Schema<GenericRecord> newAutoConsumeSchema();
 
     Schema<byte[]> newAutoProduceSchema();

pulsar-client/pom.xml

@@ -127,6 +127,11 @@
       <artifactId>commons-codec</artifactId>
     </dependency>
 
+    <dependency>
+      <groupId>com.google.guava</groupId>
+      <artifactId>guava</artifactId>
+    </dependency>
+
     <dependency>
       <groupId>com.yahoo.datasketches</groupId>
       <artifactId>sketches-core</artifactId>

pulsar-client/src/main/java/org/apache/pulsar/client/impl/PulsarClientImplementationBindingImpl.java

@@ -84,13 +84,24 @@
 import org.apache.pulsar.common.schema.SchemaInfo;
 import org.apache.pulsar.common.schema.SchemaInfoWithVersion;
 import org.apache.pulsar.common.schema.SchemaType;
-
+import com.google.common.collect.MapMaker;
+import java.util.concurrent.ConcurrentMap;
 /**
  * Helper class for class instantiations and it also contains methods to work with schemas.
  */
 @SuppressWarnings("unchecked")
 public final class PulsarClientImplementationBindingImpl implements PulsarClientImplementationBinding {
 
+    private static final PulsarClientImplementationBinding IMPLEMENTATION;
+
+    private static final ConcurrentMap<Class<?>, Schema<?>> PROTOBUF_CACHE = new MapMaker().weakKeys().makeMap();
+
+    private static final ConcurrentMap<Class<?>, Schema<?>> PROTOBUF_NATIVE_CACHE = new MapMaker().weakKeys().makeMap();
+
+    private static final ConcurrentMap<Class<?>, Schema<?>> AVRO_CACHE = new MapMaker().weakKeys().makeMap();
+
+    private static final ConcurrentMap<Class<?>, Schema<?>> JSON_CACHE = new MapMaker().weakKeys().makeMap();
+
     public <T> SchemaDefinitionBuilder<T> newSchemaDefinitionBuilder() {
         return new SchemaDefinitionBuilderImpl();
     }
@@ -208,21 +219,42 @@ public Schema<LocalDateTime> newLocalDateTimeSchema() {
     public <T> Schema<T> newAvroSchema(SchemaDefinition schemaDefinition) {
         return AvroSchema.of(schemaDefinition);
     }
+
+    public <T> Schema<T> newAvroSchema(Class<T> pojo) {
+        return (Schema<T>) AVRO_CACHE.computeIfAbsent(pojo, 
+            k -> AvroSchema.of(SchemaDefinition.builder().withPojo(pojo).build())).clone();
+    }
 
     public <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufSchema(
             SchemaDefinition schemaDefinition) {
         return ProtobufSchema.of(schemaDefinition);
     }
 
+    public <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufSchema(Class<T> clazz) {
+        return (Schema<T>) PROTOBUF_CACHE.computeIfAbsent(clazz,
+            k -> ProtobufSchema.of(SchemaDefinition.builder().withPojo(clazz).build())).clone();
+    }
+
+
     public <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufNativeSchema(
             SchemaDefinition schemaDefinition) {
         return ProtobufNativeSchema.of(schemaDefinition);
     }
 
+    public <T extends com.google.protobuf.GeneratedMessageV3> Schema<T> newProtobufNativeSchema(Class<T> clazz) {
+        return (Schema<T>) PROTOBUF_NATIVE_CACHE.computeIfAbsent(clazz, 
+            k -> ProtobufNativeSchema.of(SchemaDefinition.builder().withPojo(clazz).build())).clone();
+    }
+
     public <T> Schema<T> newJSONSchema(SchemaDefinition schemaDefinition) {
         return JSONSchema.of(schemaDefinition);
     }
 
+    public <T> Schema<T> newJSONSchema(Class<T> pojo) {
+         return (Schema<T>) JSON_CACHE.computeIfAbsent(pojo, 
+            k -> JsonSchema.of(SchemaDefinition.builder().withPojo(pojo).build())).clone();
+    }
+
     public Schema<GenericRecord> newAutoConsumeSchema() {
         return new AutoConsumeSchema();
     }