[GitHub] [solr] thelabdude commented on a change in pull request #168: SOLR-15451: SolrSchema (for SQL) should use PKI principal for request to /admin/luke

thelabdude commented on a change in pull request #168: URL: https://github.com/apache/solr/pull/168#discussion_r651965550 ## File path: solr/solr-ref-guide/src/parallel-sql-interface.adoc ## @@ -66,6 +66,14 @@ The `/sql` handler is the front end of the Parallel SQL interface.

[GitHub] [solr] thelabdude commented on a change in pull request #168: SOLR-15451: SolrSchema (for SQL) should use PKI principal for request to /admin/luke

thelabdude commented on a change in pull request #168: URL: https://github.com/apache/solr/pull/168#discussion_r649504387 ## File path: solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBSolrClient.java ## @@ -217,6 +217,15 @@ public synchronized String nextOrError(Except

[GitHub] [solr] thelabdude commented on a change in pull request #168: SOLR-15451: SolrSchema (for SQL) should use PKI principal for request to /admin/luke

thelabdude commented on a change in pull request #168: URL: https://github.com/apache/solr/pull/168#discussion_r648734114 ## File path: solr/core/src/java/org/apache/solr/handler/sql/SolrSchema.java ## @@ -90,17 +95,34 @@ public boolean isClosed() { return builder.build();

[GitHub] [solr] thelabdude commented on a change in pull request #168: SOLR-15451: SolrSchema (for SQL) should use PKI principal for request to /admin/luke

thelabdude commented on a change in pull request #168: URL: https://github.com/apache/solr/pull/168#discussion_r648629727 ## File path: solr/core/src/java/org/apache/solr/handler/sql/SolrSchema.java ## @@ -90,17 +95,34 @@ public boolean isClosed() { return builder.build();