Github user rsafonseca commented on the pull request:
https://github.com/apache/cloudstack/pull/372#issuecomment-110265493
Hi @terbolous
I'm here to help, so i'll try to break it down a bit in the coming days as
i mentioned :)
Testing this will take some time anyway, and if an issue arises with the
mysql connector change, i'll just update the PR, nothing will get stuck because
of that ;)
Btw, the change with mysql has 2 objectives:
1 - Reduce distro specific dependencies (different distros have different
package names an versions) and thus increase distro abstraction.
2 - Make the management webapp self contained (currently, if you take the
war file from this PR and put it in your own servlet engine all you need to do
is install the RPM and add /etc/cloudstack/management to the context class
loader's classpath )
Either way, do you all mean easier to review? I don't see how testing a
bunch of PRs would be faster or easier to test a single one :)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
