DefaultLazyPluginCollection unnecessarily instantiates plugins when sorting collection

jonathanshaw → changed the visibility of the branch 3520997-2 to hidden.

jonathanshaw → changed the visibility of the branch 3520997-defaultlazyplugincollection-unnecessarily-instantiates to hidden.

jonathanshaw → changed the visibility of the branch 11.x to hidden.

jonathanshaw → changed the visibility of the branch 3520997- to hidden.

Took a brief look into this and left two minor side notes.

Added another small note regards the array lookup. Other than that the code changes look good. I have no time to actually test it though.

Appears to be 1 open threads around the return. If that changes though do wonder if it will need test coverage that the exception is correctly thrown.

I'm not sure what's going on with the tests, I think the fails are unrelated.