jayunit100 / bigpetstore

[OBSOLETE] Moved to Apache BigTop !!! See you there.
http://jayunit100.github.io/bigpetstore/slides
9 stars 3 forks source link

PigDevelopmentTest: Why is there "utility code?" #6

Closed jayunit100 closed 10 years ago

jayunit100 commented 10 years ago

We shouldn't need utility code to run pig tests in IDEs. I know that eclipse will add all the files to the classpath properly when we use "mvn eclipse:eclipse" and simply "run as-> junit test..." although i forget the exact details .

Lets (1) add a comment to the " private Set getJarsForPig() {" method about why its required and then (2) remove it ASAP. :)

nsavageJVM commented 10 years ago

" will add all the files to the classpath properly " yes just removed the code and it runs ok, we do not need that method I will remove it

jayunit100 commented 10 years ago

this is now fixed. Pig doesnt get any extra jars etc... at least, its not needed for running in eclipse ide.