Re: scalastyle violation on mvn install but not on mvn package

2017-05-24 Thread Xiangyu Li
I downloaded a source code distribution of spark-2.1.0 and did the install again, and this time I did not see any warnings. I must have used some modified code before. Thank you for the help! On Tue, May 23, 2017 at 11:19 AM, Mark Hamstra wrote: > > > On Tue, May 23,

Re: scalastyle violation on mvn install but not on mvn package

2017-05-23 Thread Mark Hamstra
On Tue, May 23, 2017 at 7:48 AM, Xiangyu Li wrote: > Thank you for the answer. > > So basically it is not recommended to install Spark to your local maven > repository? I thought if they wanted to enforce scalastyle for better open > source contributions, they would have

Re: scalastyle violation on mvn install but not on mvn package

2017-05-23 Thread Xiangyu Li
Thank you for the answer. So basically it is not recommended to install Spark to your local maven repository? I thought if they wanted to enforce scalastyle for better open source contributions, they would have fixed all the scalastyle warnings. On a side note, my posts on Nabble never got

Re: scalastyle violation on mvn install but not on mvn package

2017-05-23 Thread Xiangyu Li
Thank you for the answer. So basically it is not recommended to install Spark to your local maven repository? I thought if they wanted to enforce scalastyle for better open source contributions, they would have fixed all the scalastyle warnings. On a side note, my posts on Nabble never got

Re: scalastyle violation on mvn install but not on mvn package

2017-05-17 Thread Marcelo Vanzin
scalastyle runs on the "verify" phase, which is after package but before install. On Wed, May 17, 2017 at 5:47 PM, yiskylee wrote: > ./build/mvn -Pyarn -Phadoop-2.4 -Dhadoop.version=2.4.0 -DskipTests clean > package > works, but > ./build/mvn -Pyarn -Phadoop-2.4

Re: scalastyle violation on mvn install but not on mvn package

2017-05-04 Thread Mark Hamstra
The check goal of the scalastyle plugin runs during the "verify" phase, which is between "package" and "install"; so running just "package" will not run scalastyle:check. On Thu, May 4, 2017 at 7:45 AM, yiskylee wrote: > ./build/mvn -Pyarn -Phadoop-2.4 -Dhadoop.version=2.4.0