Github user jendap commented on the pull request:
https://github.com/apache/spark/pull/7375#issuecomment-121165242
I can see you do not want to add any extra line to the pom file :-) I think
the build should primary work. Batteries included. For that reason we should
merge the lifecycle-mappings.
There are other things that would, imho, simplify the build! Namely:
1) remove "change-version-to-X.sh" scripts calling sed command
2) not running twice for integration tests - once with "-DskipTests" and
then without it (that would actually add 50 lines to pom file but improve speed
and usability)
3) workaround issues with file name lengths for some folks
... and the best of all ...
4) we can covert pom.xml to pom.scala and get a rid of the xml :-)
and even with all those done the lifecycle-mappings still belongs there.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
