Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support reading compressed geojson #200

Merged
merged 2 commits into from
Mar 6, 2018

Conversation

halfabrane
Copy link
Contributor

@halfabrane halfabrane commented Mar 6, 2018

@Perados can you check if this fixes #199 ?

Usage:

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

@codecov-io
Copy link

codecov-io commented Mar 6, 2018

Codecov Report

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

Impacted file tree graph

@@            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%) ⬆️
src/main/scala/magellan/GeoJSONRelation.scala 100% <100%> (ø) ⬆️

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.

@harsha2010 harsha2010 merged commit 1c5b99f into harsha2010:master Mar 6, 2018
@halfabrane halfabrane deleted the COMPRESSED-GEOJSON branch March 7, 2018 16:24
@Perados
Copy link
Contributor

Perados commented Apr 3, 2018

@harsha2010 It works, thank you! 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

How to read gzipped geojson?
4 participants