[
https://issues.apache.org/jira/browse/BIGTOP-1215?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sean Mackrory updated BIGTOP-1215:
----------------------------------
Attachment: 0001-BIGTOP-1215.-Refactor-Tomcat-deployment.patch
This went much smoother than I thought. This patch moves all the deployment
code to one script per component, and makes a more consistent and more obvious
distinction between the binaries and the configuration files.
I forgot to mention that one main motivation for this change is that future
releases of Sqoop 2 will provide a tooling feature that requires a separate
deployment of CATALINA_BASE from any currently running instance. Also, HttpFS
will be adding SSL support, so we'll have multiple configuration for HttpFS
just like we do for Oozie, so we might want to rename the main configuration to
tomcat-conf.http for consistency with the way Oozie does it, but that's a
separate issue...
> Refactor Tomcat deployment
> --------------------------
>
> Key: BIGTOP-1215
> URL: https://issues.apache.org/jira/browse/BIGTOP-1215
> Project: Bigtop
> Issue Type: Bug
> Reporter: Sean Mackrory
> Assignee: Sean Mackrory
> Attachments: 0001-BIGTOP-1215.-Refactor-Tomcat-deployment.patch
>
>
> Since my original changes to have Tomcat configuration stored in /etc, with
> only binaries under /usr/lib, I've had a few thoughts on how it should've
> been done better that I'd like to get in before 0.8.0, which would be the
> first release my changes are included in.
> * The code for doing the deployment should be a separate script, rather than
> a function defined separately in each entry point. This will reduce code
> duplication and prevent inconsistency, and also make it clearer for users
> following along.
> * The code in /etc only includes the /conf/ folder. There are some other
> config files that may need to be edited by users for things like HTTPS
> support. These should be stored in /etc as well and deployed at the same time.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
