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

Jarek Jarcec Cecho commented on SQOOP-910:
------------------------------------------

Thanks for the high level plan [~colin_mjj]! If you don't mind I would propose 
adding following points explicitly - I'm sure that you would cover them 
nevertheless, but I'll have better sleep if we would track them explicitly :)

* Get tools work with Jetty (they are currently using Tomcat specific code to 
bootstrap classpath)
* Do real cluster tests

In terms of development process, I'm also expecting that those changes will be 
highly intrusive and very likely we won't be able to commit them easily one by 
one. I'm not strongly opinionated here - we can create subtasks with dependent 
reviews and commit them once they are all ready  as you are proposing. 
Alternatively we can create a feature branch for you where you can develop 
until it's ready to be merged back to main branch. Even though we are not 
usually having feature branches, we've done that once with From/To refactoring. 
Since we don't have process for this and we have relatively small number of 
developers at the moment, I feel that you should be able to choose approach 
that will work best for you [~colin_mjj]. 

Regardless of what process we choose, I think that we should send a note to 
dev@ mailing list. Not every developer (myself included :() is following all 
JIRA updates and this seems as a huge change, so we should give a heads up that 
this is coming down the road.

> Sqoop2: Move to embedded jetty from tomcat
> ------------------------------------------
>
>                 Key: SQOOP-910
>                 URL: https://issues.apache.org/jira/browse/SQOOP-910
>             Project: Sqoop
>          Issue Type: Bug
>          Components: sqoop2-server
>    Affects Versions: 1.99.1
>            Reporter: Hari Shreedharan
>            Assignee: Colin Ma
>             Fix For: 2.0.0
>
>
> Embedded jetty makes it much easier to configure and replace the current jar 
> with the new one - in testing.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to