Mailing-List: contact derby-commits-help@db.apache.org; run by ezmlm
Precedence: bulk
Reply-To: "Derby Development" <derby-dev@db.apache.org>
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
From: Apache Wiki <wikidiffs@apache.org>
To: derby-commits@db.apache.org
Date: Sun, 09 Jul 2006 15:00:15 -0000
Message-ID: <20060709150015.13668.86521@ajax.apache.org>
Subject: [Db-derby Wiki] Update of "PatchListMaintenance" by KatheyMarsden

Dear Wiki user,

You have subscribed to a wiki page or wiki category on "Db-derby Wiki" for change notification.

The following page has been changed by KatheyMarsden:
http://wiki.apache.org/db-derby/PatchListMaintenance

New page:
Derby maintains a [http://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&&pid=10594&status=1&status=3&status=4&customfield_12310090=Patch+Available&sorter/field=issuekey&sorter/order=DESC&sorter/field=updated&sorter/order=DESC  Patch List] of available patches  so that developers in the community can quickly see what patches are  available and ready for review.   Open issues with the "Patch Available" checkbox marked show up on this list.
Developers wanting a patch reviewed for either commit or even just for early feedback check this box to get the attention of the community and uncheck it when review is no longer needed because of patch rework in progress or commit.

Ideally the patch available list is a small, frequently changing list of patches that are ready for  review.  To maintain this list we rely on distributed maintenance by developers who ask for review by checking the box and those doing reviews.  Patch Contributors and reviewers  please.

 1. Check the box when you want review.
 2. Make sure you are assigned to the issue if the box is checked. (You can unassign yourself later or create a subtask if you  don't plan to address the full issue.)
 3. Uncheck the box when you no longer need review because of commit or because you need to make patch changes or are just abandoning the patch.
 4. Heed the PatchAdvice Wiki page to make the patch easy to review. This will keep the list moving.


Reviewers and the general community can make periodic passes of the list to find issues to review.  In addition to improving the quality of the product, reviewing patches and keeping the list small with fast throughput will help the community grow and  become more diverse.  General maintenance also makes the list a more useful tool.  Like your mother said, leaving the list a little cleaner than you found it is normally all that is necessary.

Check the following:

|| Are there patches that you would like to review or try || Check the [http://issues.apache.org/jira/secure/ConfigureReport.jspa?filterid=12310751&mapper=components&selectedProjectId=10594&reportKey=com.atlassian.jira.plugin.system.reports%3Asinglelevelgroupby&Next=Next Patch List by component]  to see if there are issues of interest to you and review or just try them out||
|| Are there aging patches that need attention? || Check the [http://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&&pid=10594&status=1&status=3&status=4&customfield_12310090=Patch+Available&sorter/field=issuekey&sorter/order=DESC&sorter/field=updated&sorter/order=DESC  Patch List] for patches that are over 2 weeks old. Comment on the Jira issue if it looks like the patch is no longer relevant, if you think more information is required for review, or if you think the patch needs attention from someone familiar with a particular area||
|| Are all patches for Assigned issues? || Check the [http://issues.apache.org/jira/secure/ConfigureReport.jspa?filterid=12310751&mapper=assignees&selectedProjectId=10594&reportKey=com.atlassian.jira.plugin.system.reports%3Asinglelevelgroupby&Next=Next  Patch List by Owner] to see if there are Unassigned issues and inquire why they are unassigned||
|| Are there patches that are out of date? || Check  some of the older patches and see if they can be applied. Comment on the issue if the the patch can no longer be applied||
|| Are there stale patches that need to be removed?|| Check the [http://issues.apache.org/jira/secure/IssueNavigator.jspa?reset=true&&pid=10594&status=1&status=3&status=4&customfield_12310090=Patch+Available&sorter/field=issuekey&sorter/order=DESC&sorter/field=updated&sorter/order=DESC  Patch List] for issues that clearly no longer belong there and uncheck the box.  For instance patches already commited should not be on the list.||