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

Erick Erickson commented on SOLR-13952:
---------------------------------------

Thanks for letting me know about XmlOffsetCorrector.

re: one big commit.  There are over 200 (?) separate files affected, the vast 
majority of them follow the pattern:

{code}
@SuppressWarning blah blah blah
{code}

Mostly for deprecations and the like and another group for thread leaks from 
other packages that we don't control and shouldn't fail  suites because of 
thread leaks in them.

There are maybe 3-4 about matters like this. And this one (XmlOffsetCorrector) 
doesn't count since I'm going to revert it on your advice.

I'm not willing to create a huge number of tickets for this. Look at the bright 
side, at least the gradle branch won't have them (soon I hope).

The goal here is to peel this out of the Gradle build exactly to introduce 
_some_ separation of changes in the gradle branch without losing Mark's efforts 
at test improvement (or, in many cases compiler warnings).

So what do you suggest here? I'm not going to do a lot of busy work to address 
this.

> Separate out Gradle-specific code from other (mostly test) changes and commit 
> separately
> ----------------------------------------------------------------------------------------
>
>                 Key: SOLR-13952
>                 URL: https://issues.apache.org/jira/browse/SOLR-13952
>             Project: Solr
>          Issue Type: Task
>      Security Level: Public(Default Security Level. Issues are Public) 
>            Reporter: Erick Erickson
>            Assignee: Erick Erickson
>            Priority: Major
>         Attachments: fordavid.patch
>
>
> The gradle_8 branch has many changes unrelated to gradle. It would be much 
> easier to work on the gradle parts if these were separated. So here's my plan:
> - establish a branch to use for the non-gradle parts of the gradle_8 branch 
> and commit separately. For a first cut, I'll make all the changes I'm 
> confident of, and mark the others with nocommits so we can iterate and decide 
> when to merge to master and 8x.
> - create a "gradle_9" branch that hosts only the gradle changes for us all to 
> iterate on.
> I hope to have a preliminary cut at this over the weekend. 



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