[SPARK-12152][PROJECT-INFRA] Speed up Scalastyle checks by only invoking SBT once
Currently, `dev/scalastyle` invokes SBT four times, but these invocations can be replaced with a single invocation, saving about one minute of build time. Author: Josh Rosen <joshrosen@databricks.com> Closes #10151 from JoshRosen/speed-up-scalastyle.
This commit is contained in:
parent
49efd03bac
commit
80a824d36e
|
@ -17,14 +17,17 @@
|
|||
# limitations under the License.
|
||||
#
|
||||
|
||||
echo -e "q\n" | build/sbt -Pkinesis-asl -Phive -Phive-thriftserver scalastyle > scalastyle.txt
|
||||
echo -e "q\n" | build/sbt -Pkinesis-asl -Phive -Phive-thriftserver test:scalastyle >> scalastyle.txt
|
||||
# Check style with YARN built too
|
||||
echo -e "q\n" | build/sbt -Pkinesis-asl -Pyarn -Phadoop-2.2 scalastyle >> scalastyle.txt
|
||||
echo -e "q\n" | build/sbt -Pkinesis-asl -Pyarn -Phadoop-2.2 test:scalastyle >> scalastyle.txt
|
||||
|
||||
ERRORS=$(cat scalastyle.txt | awk '{if($1~/error/)print}')
|
||||
rm scalastyle.txt
|
||||
# NOTE: echo "q" is needed because SBT prompts the user for input on encountering a build file
|
||||
# with failure (either resolution or compilation); the "q" makes SBT quit.
|
||||
ERRORS=$(echo -e "q\n" \
|
||||
| build/sbt \
|
||||
-Pkinesis-asl \
|
||||
-Pyarn \
|
||||
-Phive \
|
||||
-Phive-thriftserver \
|
||||
scalastyle test:scalastyle \
|
||||
| awk '{if($1~/error/)print}' \
|
||||
)
|
||||
|
||||
if test ! -z "$ERRORS"; then
|
||||
echo -e "Scalastyle checks failed at following occurrences:\n$ERRORS"
|
||||
|
|
Loading…
Reference in a new issue