Eric Pugh

Results 586 comments of Eric Pugh

I noticed that SpellCheckComponent and QueryElevationComponent also use `getLatestSchema`... in fact, looking where `getLatestSchema` (92 times) versus `getSchema` (173 times) happens is an interesting mish mash of places.

I've looked at about half the changes... It all *seems* good... I'm going to try and run the tests before I head to bed on this branch...

Your most recent fix JUST fixed the test error I was getting... `./gradlew :solr:core:test --tests "org.apache.solr.handler.component.DistributedFacetPivotLargeTest"` passes now for me!

THis is failing... ``` ./gradlew :solr:core:test --tests "org.apache.solr.core.TestCoreContainer" ```

This all makes sense to me... @cpoerschke do you want to merge this one? Or I can.. ;-). I never could duplicate it, but it *seems* good!

This is two great suggestions... Part of my challenge is that this whole auth realm is kind of new to me, so I don't know what everything means! Going to...

Could we maybe add some docs right above the box to say what this is for? I.e, if you are using a apikey? ![image](https://user-images.githubusercontent.com/22395/195681014-d938801e-780a-43d4-9af7-e6c31aa5b96a.png) Or, should we force you to...

Change it up to add a apikey at the splainer landing page level., but in the detail have a drop down. Check on the solr custom headers box showing up...

@worleydl I think you need to commit the `customHeaders` file ;-) I don't have it!

Looks like we need both the directive and the controller! GEtting close!