+1 will commit shortly unless anyone objects to this? Here are my notes on the review.
First, ,i had to download and Export JAVA_HOME=path_to_java7_home fixed the issue. I guess there is an issue w/ gradle and scala on java 8 currently.
Anyways - building the jar worked.. I tested 3 bits of core functionality.
1) gradle tasks
This printed out all the tasks cleanly. Would be cool to customize it in the future to print out the task incantations in your README updates.
2) gradle build integrationTest
This also worked : but integrationTest SKIPPED because there was no profile. I suggest in the future we put an error message here rather than skipping all tests. After all , there is no point in running an integrationTest without a ITProfile parameter.
3) gradle clean integrationTest -P ITProfile=pig
Ran the pig flow. I saw the data in my local bps_integration after and it looks good.
So, I'll commit this shortly !
So we've preserved functionality and dropped 100s of lines of code. Awesome !
- BigPetStore now builds with gradle
- BigPetStore now supports scala
Thanks bhashit ! looking forward to more contributions to come . Will commit shortly while i port apache credentials to my new machine.