search

vegas-viz / Vegas

The missing MatPlotLib for Scala + Spark
MIT License
730 stars 98 forks source link

Adding Spark unit test #70

Closed dbtsai closed 8 years ago

dbtsai commented 8 years ago

In this PR, an unit test for Spark's dataframe's api is added. Also, RDD support is removed since RDD[Product] doesn't work in Spark due to that class RDD is invariant in type T, and when a case class is passed in, i.e., XYPair <: Product we need to make RDD[T] as RDD[+T] which we can not modify.

dbtsai commented 8 years ago

+@aishfenton

codecov-io commented 8 years ago

Current coverage is 70.00% (diff: 100%)

Merging #70 into master will increase coverage by 0.51%

@@             master        #70   diff @@
==========================================
  Files            25         25          
  Lines          1763       1760     -3   
  Methods        1759       1756     -3   
  Messages          0          0          
  Branches          4          4          
==========================================
+ Hits           1225       1232     +7   
+ Misses          538        528    -10   
  Partials          0          0

Powered by Codecov. Last update eef177e...55a2c80

aishfenton commented 8 years ago

LGTM