Notice: This Wiki is now read only and edits are no longer possible. Please see: https://gitlab.eclipse.org/eclipsefdn/helpdesk/-/wikis/Wiki-shutdown-plan for the plan.
Difference between revisions of "TM/Code Ownership"
m |
m (DSDP/TM/Code Ownership moved to TM/Code Ownership) |
||
(21 intermediate revisions by the same user not shown) | |||
Line 1: | Line 1: | ||
− | + | What do we want to achieve by explicit code ownership? | |
+ | * '''Integrity of the Design''': The owner is responsible for a solid extensible design. | ||
+ | ** People other than the owner can make bug fixes, but please let the owner know. | ||
+ | ** Don't make design-breaking changes without letting the owner know. | ||
+ | ** Typically, there is a single owner for each functionality. Multiple "two-in-a-box" owners are only possible if they work very closely together. | ||
+ | ** There may be a backup person for an owner, in case the owner is out-of-office. | ||
+ | * '''Timely Bug Triage''': The owner is responsible for reviewing (and probably reassigning) bugs of his area in a timely fashion. | ||
+ | ** Helps dispatching new bugs to component owners first. | ||
+ | ** Ownership can be for functionality that spans multiple packages (if functionality is seen by user as a single entity). | ||
+ | * '''Credit for Quality through Visibility'''. | ||
+ | ** Owners of a component should be publicly visible, so they can get the credit for good work. | ||
+ | * '''Ownership of Copyright and Intellectual Property (IP) issues'''. | ||
+ | ** Component owners are responsible for keeping the IP of their component clean. | ||
+ | <p/> | ||
+ | ==Clarification of Ambiguities== | ||
+ | * RSE Team Support (Team View, Profiles, Docs, Persistence wrt team support) -- rse.ui.view.team belongs to dmcknigh | ||
+ | ** Teamview (better called Profiles View) is its own beast - DaveD to handle this (team is different than the rest) - to be improved by 2.0 | ||
+ | * Drag&Drop, Copy&Paste -- closely tied to the views, using view-dependent Eclipse Framework --> DaveM | ||
+ | * Rse.core.filters --> DaveD (Filter creation, UI, change notification etc); filterreferences / view adapters --> DaveM | ||
+ | * EFS - not yet assigned --> Kushal, Backup DaveM | ||
+ | ==Code Ownership Table== | ||
<table border=1 cellspacing=0 cellpadding=1> | <table border=1 cellspacing=0 cellpadding=1> | ||
<tr> | <tr> | ||
Line 6: | Line 26: | ||
<th>Area</th> | <th>Area</th> | ||
<th>Plugin/Package</th> | <th>Plugin/Package</th> | ||
+ | <th>Backup owner</th> | ||
</tr> | </tr> | ||
− | <tr><td rowspan= | + | <tr> |
+ | <td rowspan=8>Dave Dykstal<br/>(IBM)</td> | ||
<td>RSE Persistency</td> | <td>RSE Persistency</td> | ||
− | <td>org.eclipse.rse.ui/persistence</td></tr> | + | <td> |
+ | org.eclipse.rse.core/persistence<br/> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.filters (persistence aspects)<br/> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.model (persistence aspects)<br/> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.persistance<br/> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.references (persistence aspects)<br/> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.internal.references (persistence aspects)<br/> | ||
+ | org.eclipse.rse.ui/filters (persistence aspects)<br/> | ||
+ | org.eclipse.rse.ui/model (persistence aspects) <br/> | ||
+ | org.eclipse.rse.ui/subsystems (persistence aspects) <br/> | ||
+ | org.eclipse.rse.ui/systems (persistence aspects) | ||
+ | </td> | ||
+ | <td>Xuan Chen</td> | ||
+ | </tr> | ||
<tr> | <tr> | ||
<td>RSE password prompt</td> | <td>RSE password prompt</td> | ||
+ | <td> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.subsystems (credential aspects)<br/> | ||
+ | org.eclipse.rse.ui/src/org.eclipse.rse.core.subsystems (credential aspects)<br/> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.dialogs (credential aspects) | ||
+ | </td> | ||
+ | <td> </td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>RSE logging</td> | ||
+ | <td> | ||
+ | org.eclipse.rse.logging - all packages | ||
+ | </td> | ||
+ | <td>Xuan Chen</td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>RSE message support</td> | ||
+ | <td> | ||
+ | /org.eclipse.rse.services/clientserver/org.eclipse.rse.services.clientserver.messages<br/> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.messages | ||
+ | </td> | ||
+ | <td> </td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>RSE filtering</td> | ||
+ | <td> | ||
+ | org.eclipse.rse.core/src/org.eclipse.rse.core.filters<br/> | ||
+ | org.eclipse.rse.ui/filters<br/> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.propertypages - classes for filters | ||
+ | </td> | ||
+ | <td> </td> | ||
+ | </tr> | ||
+ | <tr> | ||
+ | <td>Dialog Accessibility / UI Controls</td> | ||
+ | <td> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.Mnemonics<br/> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.widgets.InheritButton<br/> | ||
+ | org.eclipse.rse.ui/UI/org.eclipse.rse.ui.widgets.SystemHistoryCombo<br/> | ||
+ | org.eclipse.rse.ui - accessibility aspects | ||
+ | </td> | ||
<td> </td></tr> | <td> </td></tr> | ||
+ | <tr> | ||
+ | <td>RSE Documentation</td> | ||
+ | <td> | ||
+ | <p> | ||
+ | org.eclipse.dstore.doc.isv<br/> | ||
+ | org.eclipse.rse.doc.user<br/> | ||
+ | org.eclipse.rse.doc.isv | ||
+ | </p> | ||
+ | </td> | ||
+ | <td>Martin O</td> | ||
+ | </tr> | ||
<tr> | <tr> | ||
<td>RSE JUnit tests</td> | <td>RSE JUnit tests</td> | ||
− | <td> | + | <td> |
+ | <p> | ||
+ | org.eclipse.rse.tests<br/> | ||
+ | org.eclipse.rse.tests.framework<br/> | ||
+ | org.eclipse.rse.tests.framework.examples | ||
+ | </p></td> | ||
+ | <td>Xuan Chen</td> | ||
+ | </tr> | ||
− | <tr><td rowspan="4">Dave McKnight</td> | + | <tr><td rowspan="4">Dave McKnight<br/>(IBM)</td> |
<td>RSE dstore</td> | <td>RSE dstore</td> | ||
− | <td>org.eclipse.dstore.core<br /> | + | <td> |
+ | <p>org.eclipse.dstore.core<br /> | ||
org.eclipse.dstore.extra<br /> | org.eclipse.dstore.extra<br /> | ||
org.eclipse.rse.connectorservice.dstore<br /> | org.eclipse.rse.connectorservice.dstore<br /> | ||
Line 28: | Line 121: | ||
org.eclipse.rse.subsystems.processes.dstore<br /> | org.eclipse.rse.subsystems.processes.dstore<br /> | ||
org.eclipse.rse.subsystems.shells.dstore<br /> | org.eclipse.rse.subsystems.shells.dstore<br /> | ||
− | </td> | + | </p></td> |
+ | <td>Xuan Chen</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td>RSE services</td> | <td>RSE services</td> | ||
<td> | <td> | ||
− | org.eclipse.rse.services.files<br /> | + | <p>org.eclipse.rse.services.files<br /> |
org.eclipse.rse.services.processes<br /> | org.eclipse.rse.services.processes<br /> | ||
org.eclipse.rse.services.search<br /> | org.eclipse.rse.services.search<br /> | ||
org.eclipse.rse.services.shells<br /> | org.eclipse.rse.services.shells<br /> | ||
− | </td> | + | </p></td> |
+ | <td>Xuan Chen</td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
Line 44: | Line 139: | ||
<p>org.eclipse.rse.core<br /> | <p>org.eclipse.rse.core<br /> | ||
org.eclipse.rse.core.filters<br /> | org.eclipse.rse.core.filters<br /> | ||
− | |||
org.eclipse.rse.core.model<br /> | org.eclipse.rse.core.model<br /> | ||
org.eclipse.rse.core.subsystems<br /> | org.eclipse.rse.core.subsystems<br /> | ||
Line 52: | Line 146: | ||
</p> | </p> | ||
</td> | </td> | ||
+ | <td> </td> | ||
</tr> | </tr> | ||
<tr> | <tr> | ||
<td>RSE views</td> | <td>RSE views</td> | ||
<td> | <td> | ||
− | |||
<p>org.eclipse.rse.ui.view<br /> | <p>org.eclipse.rse.ui.view<br /> | ||
org.eclipse.rse.ui.view.monitor<br /> | org.eclipse.rse.ui.view.monitor<br /> | ||
Line 64: | Line 158: | ||
org.eclipse.rse.ui.widgets<br /> | org.eclipse.rse.ui.widgets<br /> | ||
org.eclipse.rse.ui.widgets.services<br /> | org.eclipse.rse.ui.widgets.services<br /> | ||
− | </p> | + | </p></td> |
− | </td> | + | <td> </td> |
</tr> | </tr> | ||
− | + | <tr><td rowspan=4>Kushal Munir, Xuan Chen<br/>(IBM)</td> | |
− | <tr><td rowspan=4>Kushal Munir</ | + | |
− | + | ||
− | + | ||
− | + | ||
<td>RSE Archive Handlers</td> | <td>RSE Archive Handlers</td> | ||
− | <td>org.eclipse.rse.services.clientserver.archivehandlers</td></tr> | + | <td>org.eclipse.rse.services.clientserver.archivehandlers</td> |
+ | <td> </td></tr> | ||
<tr> | <tr> | ||
<td>RSE core comm</td> | <td>RSE core comm</td> | ||
− | <td>org.eclipse.rse.core.comm</td></tr> | + | <td>org.eclipse.rse.core.comm</td> |
+ | <td> </td></tr> | ||
<tr> | <tr> | ||
<td>RSE search</td> | <td>RSE search</td> | ||
− | <td>org.eclipse.rse.files.ui.search</td></tr> | + | <td> |
+ | <p>org.eclipse.rse.files.ui.search<br /> | ||
+ | org.eclipse.rse.ui.view.search | ||
+ | </p></td> | ||
+ | <td> </td></tr> | ||
+ | <tr> | ||
+ | <td>RSE file encodings</td> | ||
+ | <td>(multiple)</td> | ||
+ | <td> </td></tr> | ||
− | <tr><td rowspan= | + | <tr><td rowspan=1>Uwe Stieber<br/>(Wind River)</td> |
+ | <td>RSE New Connection Wizard</td> | ||
+ | <td>org.eclipse.rse.ui/org.eclipse.rse.ui.wizards<br/> | ||
+ | org.eclipse.rse.ui/org.eclipse.rse.ui.wizards.newconnection<br/> | ||
+ | org.eclipse.rse.ui/org.eclipse.rse.ui.wizards.registries<br/> | ||
+ | </td> | ||
+ | <td>Kushal Munir</td></tr> | ||
+ | <tr> | ||
+ | |||
+ | <tr><td rowspan=9>Martin Oberhuber<br/>(Wind River)</td> | ||
<tr> | <tr> | ||
<td>RSE ssh</td> | <td>RSE ssh</td> | ||
− | <td>org.eclipse.rse.connectorservice.ssh<br/> | + | <td> |
+ | <p>org.eclipse.rse.connectorservice.ssh<br/> | ||
org.eclipse.rse.services.ssh<br/> | org.eclipse.rse.services.ssh<br/> | ||
org.eclipse.rse.ssh-feature<br/> | org.eclipse.rse.ssh-feature<br/> | ||
org.eclipse.rse.subsystems.files.ssh<br/> | org.eclipse.rse.subsystems.files.ssh<br/> | ||
− | org.eclipse.rse.subsystems.shells.ssh</ | + | org.eclipse.rse.subsystems.shells.ssh |
− | + | </p></td> | |
− | + | <td> </td></tr> | |
− | <td> | + | |
− | + | ||
− | + | ||
<tr> | <tr> | ||
<td>RSE local</td> | <td>RSE local</td> | ||
− | <td>org.eclipse.rse.connectorservice.local<br/> | + | <td> |
+ | <p>org.eclipse.rse.connectorservice.local<br/> | ||
org.eclipse.rse.local-feature<br/> | org.eclipse.rse.local-feature<br/> | ||
org.eclipse.rse.services.local<br/> | org.eclipse.rse.services.local<br/> | ||
org.eclipse.rse.subsystems.files.local<br/> | org.eclipse.rse.subsystems.files.local<br/> | ||
org.eclipse.rse.subsystems.processes.local<br/> | org.eclipse.rse.subsystems.processes.local<br/> | ||
− | org.eclipse.rse.subsystems.shells.local</td></tr> | + | org.eclipse.rse.subsystems.shells.local |
+ | </p></td> | ||
+ | <td>Kushal Munir</td></tr> | ||
<tr> | <tr> | ||
<td>RSE examples</td> | <td>RSE examples</td> | ||
− | <td> | + | <td> |
− | org.eclipse.rse.examples.daytime<br/> | + | <p>org.eclipse.rse.examples.daytime<br/> |
org.eclipse.rse.examples.tutorial<br/> | org.eclipse.rse.examples.tutorial<br/> | ||
− | + | </p></td> | |
+ | <td>Dave Dykstal</td></tr> | ||
<tr> | <tr> | ||
<td>RSE content assist</td> | <td>RSE content assist</td> | ||
− | <td>org.eclipse.rse.shells.ui/org.eclipse.rse.shells.ui.view </td></tr> | + | <td>org.eclipse.rse.shells.ui/org.eclipse.rse.shells.ui.view </td> |
+ | <td>Dave McKnight</td></tr> | ||
<tr> | <tr> | ||
− | <td>RSE nightly builds</td> | + | <td> |
− | <td>org.eclipse.rse.build<br/> | + | <p>RSE nightly builds,<br/> |
+ | Legal docs (about files, licenses),<br/> | ||
+ | Update Site,<br/> | ||
+ | build notes | ||
+ | </p></td> | ||
+ | <td> | ||
+ | <p>org.eclipse.rse.build<br/> | ||
org.eclipse.rse.core-feature<br/> | org.eclipse.rse.core-feature<br/> | ||
org.eclipse.rse.dstore-feature<br/> | org.eclipse.rse.dstore-feature<br/> | ||
− | org.eclipse.rse. | + | org.eclipse.rse.efs-feature<br/> |
+ | org.eclipse.rse.examples-feature<br/> | ||
+ | org.eclipse.rse.remotecdt-feature<br/> | ||
org.eclipse.rse.sdk<br/> | org.eclipse.rse.sdk<br/> | ||
org.eclipse.rse.sdk-feature<br/> | org.eclipse.rse.sdk-feature<br/> | ||
− | org.eclipse.rse.updatesite</td></tr> | + | org.eclipse.rse.releng.infocenter<br/> |
+ | org.eclipse.rse.updatesite | ||
+ | </p></td> | ||
+ | <td> | ||
+ | <p>Ted Williams,<br/> | ||
+ | Dave Dykstal | ||
+ | </p></td></tr> | ||
+ | <tr> | ||
+ | <td>Third Party Libs<br/>(Jakarta Commons Net, ORO)</td> | ||
+ | <td> | ||
+ | <p>org.eclipse.tm.core/thirdparty/* | ||
+ | </p></td> | ||
+ | <td> | ||
+ | <p>Dave Dykstal | ||
+ | </p></td></tr> | ||
<tr> | <tr> | ||
<td>RSE manual tests</td> | <td>RSE manual tests</td> | ||
− | <td>org.eclipse.rse.tests.manual</td></tr> | + | <td>org.eclipse.rse.tests.manual</td> |
+ | <td> </td></tr> | ||
− | <tr><td rowspan= | + | <tr> |
+ | <tr><td rowspan=2>Javier Montalvo<br/>(Symbian)</td> | ||
<td>Discovery</td> | <td>Discovery</td> | ||
− | <td>(org.eclipse.tm.core) discovery/*</td></tr> | + | <td>(org.eclipse.tm.core) discovery/*</td> |
+ | <td> </td></tr> | ||
− | <tr><td rowspan=1> | + | <td>RSE ftp</td> |
+ | <td> | ||
+ | <p>org.eclipse.rse.ftp-feature<br/> | ||
+ | org.eclipse.rse.services.files.ftp<br/> | ||
+ | org.eclipse.rse.subsystems.files.ftp | ||
+ | </p></td> | ||
+ | <td> | ||
+ | <p>Dave McKnight,<br/> | ||
+ | Martin Oberhuber | ||
+ | </p></td></tr> | ||
+ | |||
+ | <tr><td rowspan=1>Michael Scharf<br/>(Wind River)</td> | ||
<td>Terminalview</td> | <td>Terminalview</td> | ||
− | <td>(org.eclipse.tm.core) terminal/*</td></tr> | + | <td>(org.eclipse.tm.core) terminal/*</td> |
+ | <td>Ted Williams</td></tr> | ||
+ | <tr><td rowspan=1>Ewa Matejska<br/>(ACCESS)</td> | ||
+ | <td>CDT Remote Launch</td> | ||
+ | <td> | ||
+ | <p>org.eclipse.rse.remotecdt | ||
+ | </p></td> | ||
+ | <td>Martin Oberhuber</td></tr> | ||
+ | |||
+ | <tr><td rowspan=1>Radoslav Gerganov<br/>(ProSyst)</td> | ||
+ | <td>WinCE subsystems</td> | ||
+ | <td> | ||
+ | <p>org.eclipse.tm.core/wince/* | ||
+ | </p></td> | ||
+ | <td>Martin Oberhuber</td></tr> | ||
+ | |||
+ | |||
+ | <tr><td rowspan=1>Anna Dushistova<br/>(MontaVista)</td> | ||
+ | <td>RSE terminal subsystems</td> | ||
+ | <td> | ||
+ | <p>org.eclipse.rse.terminal.* | ||
+ | </p></td> | ||
+ | <td>Martin Oberhuber</td></tr> | ||
</table> | </table> |
Latest revision as of 11:22, 14 December 2010
What do we want to achieve by explicit code ownership?
- Integrity of the Design: The owner is responsible for a solid extensible design.
- People other than the owner can make bug fixes, but please let the owner know.
- Don't make design-breaking changes without letting the owner know.
- Typically, there is a single owner for each functionality. Multiple "two-in-a-box" owners are only possible if they work very closely together.
- There may be a backup person for an owner, in case the owner is out-of-office.
- Timely Bug Triage: The owner is responsible for reviewing (and probably reassigning) bugs of his area in a timely fashion.
- Helps dispatching new bugs to component owners first.
- Ownership can be for functionality that spans multiple packages (if functionality is seen by user as a single entity).
- Credit for Quality through Visibility.
- Owners of a component should be publicly visible, so they can get the credit for good work.
- Ownership of Copyright and Intellectual Property (IP) issues.
- Component owners are responsible for keeping the IP of their component clean.
<p/>
Clarification of Ambiguities
- RSE Team Support (Team View, Profiles, Docs, Persistence wrt team support) -- rse.ui.view.team belongs to dmcknigh
- Teamview (better called Profiles View) is its own beast - DaveD to handle this (team is different than the rest) - to be improved by 2.0
- Drag&Drop, Copy&Paste -- closely tied to the views, using view-dependent Eclipse Framework --> DaveM
- Rse.core.filters --> DaveD (Filter creation, UI, change notification etc); filterreferences / view adapters --> DaveM
- EFS - not yet assigned --> Kushal, Backup DaveM
Code Ownership Table
Owner | Area | Plugin/Package | Backup owner |
---|---|---|---|
Dave Dykstal (IBM) |
RSE Persistency |
org.eclipse.rse.core/persistence |
Xuan Chen |
RSE password prompt |
org.eclipse.rse.core/src/org.eclipse.rse.core.subsystems (credential aspects) |
||
RSE logging |
org.eclipse.rse.logging - all packages |
Xuan Chen | |
RSE message support |
/org.eclipse.rse.services/clientserver/org.eclipse.rse.services.clientserver.messages |
||
RSE filtering |
org.eclipse.rse.core/src/org.eclipse.rse.core.filters |
||
Dialog Accessibility / UI Controls |
org.eclipse.rse.ui/UI/org.eclipse.rse.ui.Mnemonics |
||
RSE Documentation |
org.eclipse.dstore.doc.isv |
Martin O | |
RSE JUnit tests |
org.eclipse.rse.tests |
Xuan Chen | |
Dave McKnight (IBM) |
RSE dstore |
org.eclipse.dstore.core |
Xuan Chen |
RSE services |
org.eclipse.rse.services.files |
Xuan Chen | |
RSE core model |
org.eclipse.rse.core |
||
RSE views |
org.eclipse.rse.ui.view |
||
Kushal Munir, Xuan Chen (IBM) |
RSE Archive Handlers | org.eclipse.rse.services.clientserver.archivehandlers | |
RSE core comm | org.eclipse.rse.core.comm | ||
RSE search |
org.eclipse.rse.files.ui.search |
||
RSE file encodings | (multiple) | ||
Uwe Stieber (Wind River) |
RSE New Connection Wizard | org.eclipse.rse.ui/org.eclipse.rse.ui.wizards org.eclipse.rse.ui/org.eclipse.rse.ui.wizards.newconnection |
Kushal Munir |
Martin Oberhuber (Wind River) |
|||
RSE ssh |
org.eclipse.rse.connectorservice.ssh |
||
RSE local |
org.eclipse.rse.connectorservice.local |
Kushal Munir | |
RSE examples |
org.eclipse.rse.examples.daytime |
Dave Dykstal | |
RSE content assist | org.eclipse.rse.shells.ui/org.eclipse.rse.shells.ui.view | Dave McKnight | |
RSE nightly builds, |
org.eclipse.rse.build |
Ted Williams, | |
Third Party Libs (Jakarta Commons Net, ORO) |
org.eclipse.tm.core/thirdparty/* |
Dave Dykstal | |
RSE manual tests | org.eclipse.rse.tests.manual | ||
Javier Montalvo (Symbian) |
Discovery | (org.eclipse.tm.core) discovery/* | RSE ftp |
org.eclipse.rse.ftp-feature |
Dave McKnight, | </tr>
Michael Scharf (Wind River) |
Terminalview | (org.eclipse.tm.core) terminal/* | Ted Williams |
Ewa Matejska (ACCESS) |
CDT Remote Launch |
org.eclipse.rse.remotecdt |
Martin Oberhuber |
Radoslav Gerganov (ProSyst) |
WinCE subsystems |
org.eclipse.tm.core/wince/* |
Martin Oberhuber |
Anna Dushistova (MontaVista) |
RSE terminal subsystems |
org.eclipse.rse.terminal.* |
Martin Oberhuber |