harsha2010 / magellan

Geo Spatial Data Analytics on Spark
Apache License 2.0
534 stars 149 forks source link

Add support for reading Null Shapes #225

Open kd35a opened 6 years ago

kd35a commented 6 years ago

Content of the test_null shapefile:

wkt_geom NAME (10)
Point (3 3) has geom
NULL null geom
codecov-io commented 6 years ago

Codecov Report

Merging #225 into master will decrease coverage by 0.14%. The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #225      +/-   ##
==========================================
- Coverage   90.16%   90.01%   -0.15%     
==========================================
  Files          52       54       +2     
  Lines        1789     1803      +14     
  Branches      124      123       -1     
==========================================
+ Hits         1613     1623      +10     
- Misses        176      180       +4
Impacted Files Coverage Δ
src/main/scala/magellan/Shape.scala 66.66% <ø> (+8.13%) :arrow_up:
src/main/scala/magellan/io/ShapeReader.scala 92.3% <100%> (+1.92%) :arrow_up:
src/main/scala/magellan/NullShape.scala 37.5% <37.5%> (ø)
...cala/org/apache/spark/sql/types/NullShapeUDT.scala 54.54% <54.54%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update aa9021e...737fb3f. Read the comment docs.