dsmiley commented on pull request #221: URL: https://github.com/apache/solr/pull/221#issuecomment-881417808
I proposed to Nazerke that she file this PR and that it was too simple to warrant a JIRA issue as well. The issue description should have been filled to justify the PR's existence/purpose but wasn't. Basically all operations of ConfigSetService that read/write ought to throw an IOException and most do but not this one, which I think was simply an oversight. Thus if you call this, perhaps delegating, and if you call something that might throw IOException, then you have to wrap it. Other methods ought to throw IOException as well but don't declare it now: * loadConfigSetFlags That's it, as far as I can see. -- 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: issues-unsubscr...@solr.apache.org 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
