Skip to content
This repository has been archived by the owner on Sep 1, 2022. It is now read-only.

Build warnings #519

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Build warnings #519

wants to merge 1 commit into from

Conversation

peller
Copy link
Collaborator

@peller peller commented Sep 28, 2016

No description provided.

@peller
Copy link
Collaborator Author

peller commented Sep 28, 2016

@poplav: Some other (non-fatal) warnings and errors to look at, mostly from Scala:

16/09/22 16:52:23 [ERROR] o.a.s.u.Utils - Uncaught exception in thread driver-heartbeater java.io.IOException: java.lang.ClassCastException: cannot assign instance of scala.collection.immutable.HashMap$SerializationProxy to field org.apache.spark.executor.TaskMetrics._accumulatorUpdates of type scala.collection.immutable.Map in instance of org.apache.spark.executor.TaskMetrics

also

com.fasterxml.jackson.core.JsonParseException: Unrecognized token 'not': was expecting 'null', 'true', 'false' or NaN (includes a stack trace)

That section includes lots of [WARN] and [ERROR] conditions which may be part of the test. Perhaps they should be redirected away from the logs if the output is considered normal?

@poplav
Copy link
Contributor

poplav commented Sep 28, 2016

So far I found: In kernel-scala/src/main/scala/declarativewidgets/util/Explore.scala changing:
val valDefTrees = requestTrees collect {case c: ValDef => c} to
val valDefTrees = requestTrees.map(_.asInstanceOf[ValDef]) gets rid of the one warning left.

@peller peller force-pushed the build-warnings branch 4 times, most recently from 7a1f583 to 62511c0 Compare September 30, 2016 22:51
@peller peller changed the title [WIP] Build warnings Build warnings Oct 1, 2016
@peller
Copy link
Collaborator Author

peller commented Oct 1, 2016

Refs #517

@peller
Copy link
Collaborator Author

peller commented Oct 4, 2016

@lbustelo ready for review. Switch to FF for now, perhaps leave #517 open until we identify an issue for chrome/linux

(c) Copyright IBM Corp. 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants