[ 
https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-643?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12869517#action_12869517
 ] 

Delos Dai commented on GERONIMODEVTOOLS-643:
--------------------------------------------

Complete item 3 in revision #946542 and #946558

> Refactor GEP 3.0 code to decouple various server adapters
> ---------------------------------------------------------
>
>                 Key: GERONIMODEVTOOLS-643
>                 URL: 
> https://issues.apache.org/jira/browse/GERONIMODEVTOOLS-643
>             Project: Geronimo-Devtools
>          Issue Type: Improvement
>          Components: eclipse-plugin
>    Affects Versions: 3.0
>            Reporter: Delos Dai
>            Assignee: Delos Dai
>             Fix For: 3.0
>
>
> GEP 3.0 will contain server adapters of all versions, including 
> v1.1,v2.0,v2.1,v2.2,v3.0. It makes current GEP code much complicated. 
> IMO, we have following guideline of GEP structure design
> 1) "org.apache.geronimo.st.core" and ""org.apache.geronimo.st.ui" has common 
> functions used by various adapters. These two plug-ins shouln't depend on any 
> version specific adapter plug-in, such as 
> "org.apache.geronimo.runtime.vXX","org.apache.geronimo.st.vXX.core" and 
> "org.apache.geronimo.st.vXX.ui" 
> 2) "org.apache.geronimo.j2ee.v11.jaxbmodel" is dedicated to v11 adapter, 
> while "org.apache.geronimo.jee.v21.jaxbmodel" is for v20,v21,v22 adapter. So 
> far, no jaxb model updated for v30, so v30 adapter still depends on  
> "org.apache.geronimo.jee.v21.jaxbmodel"
> 3 )Since v20,v21 and v22 adapters all depends on v21  jaxb model , they 
> shared operations on the model. Currently, we put all their shared  
> operations in "org.apache.geronimo.st.v21.core" and 
> ""org.apache.geronimo.st.v21.ui". Both plug-ins of v20 and v21 adapters 
> depend on them.
> 4) For v11 adapter, it's the only  adapter for JEE 4 and it depends on v11 
> jaxb model. Except common functions in "org.apache.geronimo.st.core", it 
> doesn't have much in common with other adapter. 
> 5) For v30 adapter, it's the only adapter for JavaEE 6. Currently, since it 
> still uses v21 jaxb model, it needs operations on the model. As a result, V30 
> plug-in still depends on "org.apache.geronimo.st.v21.core" and 
> ""org.apache.geronimo.st.v21.ui". In future, once updated jaxb model for v30 
> is added, we will remove v21.core and v21.ui from its dependency list.
> Unfortunately, current GEP code doesn't strictly obey the guideline above. So 
> I did some refactoring work for GEP 3.0 code. I won't commit the change until 
> we get an agreement on this.
> I hope all of us are able to have the same undertstanding on GEP structure. 
> It will avoid potential confusion in future.  
> Thanks!

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to