madrob commented on a change in pull request #165:
URL: https://github.com/apache/solr/pull/165#discussion_r646648410
##########
File path: solr/core/src/java/org/apache/solr/api/ApiBag.java
##########
@@ -301,19 +301,18 @@ public static SpecProvider constructSpec(PluginInfo info)
{
Object specObj = info == null ? null : info.attributes.get("spec");
if (specObj == null) specObj = "emptySpec";
if (specObj instanceof Map) {
- @SuppressWarnings({"rawtypes"})
- Map map = (Map) specObj;
+ assert false : "got a map from a string";
Review comment:
info.attributes is a Map<String,String> so we should never be getting a
specObj that is a Map. However, the code exists and I don't necessarily trust
out type safety everywhere, so I wanted to give our tests a chance to prove me
wrong.
--
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.
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
