stillalex commented on PR #2101: URL: https://github.com/apache/solr/pull/2101#issuecomment-1847923908
the failing test was caused by my change, so I had to reduce the open count check to this metric only. I had extended it eagerly to all 3 metrics for some idea of consistency, but that didn't play well with the PlacementPluginIntegrationTest. I think there is some underlying expectation in the placement plugin test to have access to the metrics even if the core is not actively open and my change made it so there are no metrics available which broke the test. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
