[ https://issues.apache.org/jira/browse/SOLR-8392?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17128617#comment-17128617 ]
Erick Erickson commented on SOLR-8392: -------------------------------------- Kudos for tackling this. This kind of thing is why I'm afraid to do much code rearranging while getting the warnings out, each one is going to require thought.... > SolrParam.get(String) returns String and shouldn't be used in other > instanceof checks > ------------------------------------------------------------------------------------- > > Key: SOLR-8392 > URL: https://issues.apache.org/jira/browse/SOLR-8392 > Project: Solr > Issue Type: Bug > Reporter: Mike Drob > Assignee: Mike Drob > Priority: Major > Fix For: 7.0 > > Attachments: SOLR-8392.patch, SOLR-8392.patch > > > There's a couple of places where we declare the return type of > solrParams.get() as an Object and then do instanceof checks for other types. > Since we know it will be a String, we can simplify this logic in several > places. -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org