[MAHOUT-1563] Clean up WARNINGs during build

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 0.9
• Fix Version/s: 0.10.0
• Component/s: build-classic
• Labels: DSL scala
• Assignee: Stevo Slavić
• Reporter: Andrew Musselman
• Created: 27/May/14 00:34
• Updated: 13/Apr/15 09:57
• Resolved: 29/Mar/15 22:03

Description

We need to clean up warnings in the maven logs. They seem to have piled up recently; some are about scala lib version conflicts, some are about deprecated APIs, some are about code style.

Some may be fine for now but extra warnings in build logs feels like bad hygiene to me.

Some examples:

[WARNING] Expected all dependencies to require Scala version: 2.10.3
[WARNING] com.twitter:chill_2.10:0.3.1 requires scala version: 2.10.0
[WARNING] Multiple versions of scala libraries detected!

[WARNING] /home/akm/mahout/spark/src/main/scala/org/apache/mahout/sparkbindings/drm/CheckpointedDrmBase.scala:73: warning: a pure expression does nothing in statement position; you may be omitting necessary parentheses
[INFO] this

[WARNING] Expected all dependencies to require Scala version: 2.10.3
[WARNING] org.apache.mahout:mahout-math-scala:1.0-SNAPSHOT requires scala version: 2.10.3
[WARNING] org.scalatest:scalatest_2.10:2.0 requires scala version: 2.10.0
[WARNING] Multiple versions of scala libraries detected!

[WARNING] /home/akm/mahout/math-scala/src/main/scala/org/apache/mahout/math/scalabindings/package.scala:132: warning: non-variable type argument Double in type pattern Iterable[Double] is unchecked since it is eliminated by erasure
[INFO] case t: Iterable[Double] => t.toArray

[WARNING] /home/akm/mahout/examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java: Some input files use or override a deprecated API.
[WARNING] /home/akm/mahout/examples/src/main/java/org/apache/mahout/clustering/syntheticcontrol/kmeans/Job.java: Recompile with -Xlint:deprecation for details.

Issue Links

• is related to: MAHOUT-1654 Migrate from Maven to SBT