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

Downgrade Avro version #2837

Merged
merged 1 commit into from Dec 5, 2018

Conversation

@pomadchin
Copy link
Member

commented Dec 3, 2018

Overview

This PR downgrades Avro version down to 1.7.7 to correspond Hadoop version.

TODO

  • Check on AWS EMR cluster
  • Read the old catalog
  • CQ 18476

@pomadchin pomadchin force-pushed the pomadchin:upd/avro-downgrade branch from 6218da9 to 86f19d5 Dec 4, 2018

@pomadchin

This comment has been minimized.

Copy link
Member Author

commented Dec 4, 2018

Did a test ingest basing on the data from #2819

Catalog path: s3://geotrellis-test/issue-2819/catalog
Layer name: china5

@pomadchin pomadchin added the needs-cq label Dec 4, 2018

@pomadchin pomadchin added this to the 3.0 milestone Dec 4, 2018

@pomadchin

This comment has been minimized.

Copy link
Member Author

commented Dec 4, 2018

Test used to read the old catalog:

lazy val attributeStore = new S3AttributeStore("geotrellis-test", "issue-2819/catalog")
val lreader = S3LayerReader(attributeStore)

val st: Raster[MultibandTile] =
  lreader.read[SpatialKey, MultibandTile, TileLayerMetadata[SpatialKey]](LayerId("china2", 7)).stitch

val md: TileLayerMetadata[SpatialKey] =
  attributeStore.readMetadata[TileLayerMetadata[SpatialKey]](LayerId("china2", 7))

GeoTiff(st, md.crs).write("/tmp/china2.tiff")

@pomadchin pomadchin removed the needs-cq label Dec 5, 2018

@pomadchin pomadchin merged commit 7d5fb93 into locationtech:master Dec 5, 2018

1 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
ip-validation
Details

@pomadchin pomadchin deleted the pomadchin:upd/avro-downgrade branch Dec 12, 2018

echeipesh added a commit that referenced this pull request Dec 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.