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

Build warnings#519

Open
peller wants to merge 1 commit into
jupyter:masterfrom
peller:build-warnings
Open

Build warnings#519
peller wants to merge 1 commit into
jupyter:masterfrom
peller:build-warnings

Conversation

@peller

@peller peller commented Sep 28, 2016

Copy link
Copy Markdown
Collaborator

No description provided.

@peller

peller commented Sep 28, 2016

Copy link
Copy Markdown
Collaborator Author

@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

poplav commented Sep 28, 2016

Copy link
Copy Markdown
Contributor

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

peller commented Oct 1, 2016

Copy link
Copy Markdown
Collaborator Author

Refs #517

@peller

peller commented Oct 4, 2016

Copy link
Copy Markdown
Collaborator Author

@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