bszabo97 commented on code in PR #1649:
URL: https://github.com/apache/solr/pull/1649#discussion_r1210279607


##########
solr/core/src/java/org/apache/solr/handler/admin/api/SchemaInfoAPI.java:
##########
@@ -31,18 +38,29 @@
  * <p>This API (GET /v2/collections/collectionName/schema) is analogous to the 
v1
  * /solr/collectionName/schema API.
  */
-public class SchemaInfoAPI {
-  private final SchemaHandler schemaHandler;
+public class SchemaInfoAPI extends JerseyResource {
 
-  public SchemaInfoAPI(SchemaHandler schemaHandler) {
-    this.schemaHandler = schemaHandler;
+  private SolrCore solrCore;
+
+  @Inject
+  public SchemaInfoAPI(SolrCore solrCore) {
+    this.solrCore = solrCore;
+  }
+
+  @GET
+  @Path("/schema")
+  @Produces({MediaType.APPLICATION_JSON, MediaType.APPLICATION_ATOM_XML, 
BINARY_CONTENT_TYPE_V2})

Review Comment:
   Yes you are right I wanted to use `MediaType.APPLICATION_XML` here. The IDEA 
got me on this one...



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@solr.apache.org
For additional commands, e-mail: issues-h...@solr.apache.org

Reply via email to