Github user willb commented on the pull request:
https://github.com/apache/spark/pull/1323#issuecomment-55345856
Hi @jkbradley; thanks for taking a look. Here are the warnings as I see
them when compiling tests on the immediate ancestor of my branch, which is
56e009d (I'm running on OS X 10.9 in this case):
```
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:38:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] This can be achieved by adding the import clause 'import
scala.language.postfixOps'
[warn] or by setting the compiler option -language:postfixOps.
[warn] See the Scala docs for value scala.language.postfixOps for a
discussion
[warn] why the feature should be explicitly enabled.
[warn] implicit val defaultTimeout = timeout(10000 millis)
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:99:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] preGCTester.assertCleanup()(timeout(1000 millis))
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:117:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] preGCTester.assertCleanup()(timeout(1000 millis))
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:134:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] preGCTester.assertCleanup()(timeout(1000 millis))
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:156:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] preGCTester.assertCleanup()(timeout(1000 millis))
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:187:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] preGCTester.assertCleanup()(timeout(1000 millis))
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ContextCleanerSuite.scala:290:
postfix operator millis should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(waitTimeout, interval(100 millis)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/rdd/AsyncRDDActionsSuite.scala:138:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] failAfter(10 seconds) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/rdd/AsyncRDDActionsSuite.scala:174:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] failAfter(10 seconds) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:42:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:42:
postfix operator milliseconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:56:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:56:
postfix operator milliseconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:75:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:75:
postfix operator milliseconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:89:
postfix operator seconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
[warn] ^
[warn]
/Users/willb/Documents/src/scala/alternate-spark/core/src/test/scala/org/apache/spark/ui/UISuite.scala:89:
postfix operator milliseconds should be enabled
[warn] by making the implicit value scala.language.postfixOps visible.
[warn] eventually(timeout(10 seconds), interval(50 milliseconds)) {
```
Moving the import to within the test class (as in my patch) eliminated the
warnings.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
