halfabrane commented 6 years ago

@Perados can you check if this fixes #199 ?

Usage:

spark.read.format("magellan")
  .option("type", "geojson")
  .load("$path_to_geojson.gz")

codecov-io commented 6 years ago

Codecov Report

Merging #200 into master will increase coverage by 0.05%. The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #200      +/-   ##
=========================================
+ Coverage   89.74%   89.8%   +0.05%     
=========================================
  Files          50      50              
  Lines        1745    1755      +10     
  Branches      117     128      +11     
=========================================
+ Hits         1566    1576      +10     
  Misses        179     179

Impacted Files	Coverage Δ
...ain/scala/magellan/mapreduce/WholeFileReader.scala	`97.05% <100%> (+0.9%)`	:arrow_up:
src/main/scala/magellan/GeoJSONRelation.scala	`100% <100%> (ø)`	:arrow_up:

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 a59de0b...3330f48. Read the comment docs.

Perados commented 6 years ago

@harsha2010 It works, thank you! 🚀

harsha2010 / magellan

Support reading compressed geojson #200

Codecov Report