[ 
https://issues.apache.org/jira/browse/SOLR-10778?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17106388#comment-17106388
 ] 

Erick Erickson commented on SOLR-10778:
---------------------------------------

[~atris] [~gus]  Many thanks for volunteering. I assigned each of you a 
directory, but there's no reason at all you need to be obligated to do that 
particular one. Please feel absolutely free to pick some other directory and 
create a sub-task for yourself. All I'm really doing here is using sub-tasks to 
divvy up the work.

It'll be a few days before I'm ready to tackle any more, I've got a couple of 
other cleanups to get done first to preserve some of the work for SOLR-10810...

> Address precommit WARNINGS
> --------------------------
>
>                 Key: SOLR-10778
>                 URL: https://issues.apache.org/jira/browse/SOLR-10778
>             Project: Solr
>          Issue Type: Improvement
>          Components: clients - java
>    Affects Versions: 4.6
>            Reporter: Andrew Musselman
>            Assignee: Erick Erickson
>            Priority: Minor
>         Attachments: dated-warnings, dated-warnings.log, notclosed.txt
>
>
> During precommit we are seeing lots of warnings. Im turning this into an 
> umbrella issue about getting precommit warnings out of the code in general. 
> Yes, this will take a while.
> See SOLR-10809 for getting all warnings out of solr/core. I want to 
> selectively have precommit fail when "some part" of the code is clean so we 
> don't backslide, and solr/core was the finest granularity I could see how to 
> change.
> If you read more of the comments here, you can see that there are some 
> serious code refactoring that could be done. I'm electing to simply 
> SuppressWarnings rather than re-arrange code at this point whenever the code 
> is tricky. If anyone goes back in and tries to clean the code up, then can 
> remove the annotation(s).



--
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

Reply via email to