You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
If you use Spark 1.2.0, this is the code that can be removed:
.filter(!_.contains("boundingBoxCoordinates")) // TODO(vida): Remove this workaround when SPARK-3390 is fixed.
If you remove it for Spark 1.1.0, Collect.java won't break when run, but ExamineAndTrain.scala will do, with a "scala.MatchError: StructType(List())" exception. It will be caused by the "boundingBoxCoordinates" json entries, as Spark 1.1.0 doesn't handle them properly.
The text was updated successfully, but these errors were encountered:
MiguelPeralvo
added a commit
to MiguelPeralvo/reference-apps
that referenced
this issue
Jan 4, 2015
Fixes [issue databricks#50: Pending TODO can be completed: SPARK-3390 was fixed] (databricks#50). I've tested it in Spark 1.1.0 and 1.2.0 and it works, as expected.
MiguelPeralvo
changed the title
twitter_classifier/Collect.scala: Pending TODO can be completed: SPARK-3390 was fixed.
twitter_classifier/Collect.scala: Pending TODO can be completed: SPARK-3390 was fixed in Spark 1.2.0.
Jan 4, 2015
Line 42 of reference-apps/twitter_classifier/scala/src/main/scala/com/databricks/apps/twitter_classifier/Collect.scala can now be safely removed, as SPARK-3390 was fixed in pull request #2364 for Apache 1.2.0.
If you use Spark 1.2.0, this is the code that can be removed:
If you remove it for Spark 1.1.0, Collect.java won't break when run, but ExamineAndTrain.scala will do, with a "scala.MatchError: StructType(List())" exception. It will be caused by the "boundingBoxCoordinates" json entries, as Spark 1.1.0 doesn't handle them properly.
The text was updated successfully, but these errors were encountered: