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.

Sign up for GitHub

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

Jump to bottom

Update Samples & Tutorials to Spatial Framework for Hadoop v2 and Geometry v2 #66

Open
randallwhitman opened this issue Dec 4, 2017 · 3 comments
Open

Update Samples & Tutorials to Spatial Framework for Hadoop v2 and Geometry v2 #66

randallwhitman opened this issue Dec 4, 2017 · 3 comments
Assignees
@randallwhitman @smambrose
Labels
bug enhancement
Milestone
v2.1

Comments

@randallwhitman
Copy link
Contributor

Where applicable need to use new names [Un]enclosedEsriJsonInputFormat and EsriJsonSerDe.
Review direct Geometry calls relating to JSON to update exceptions checked.
Geometry v2 also implies Jackson v2.
Mismatch with the latest versions creates confusion and leads to issues e.g. #65.
@randallwhitman randallwhitman added this to the v2.1 milestone Dec 4, 2017
This was referenced Dec 4, 2017
Closed
Closed
@randallwhitman
Copy link
Contributor Author

randallwhitman commented Dec 29, 2017 •
edited
Loading

Apparently I committed JSON-naming changes to branch without ever merging - it may be that the changes hadn't gotten tested.
[update] No - that was merged in #58, but not sure how thoroughly it was tested, if the pre-built JAR files in the lib directory were not updated to match.

This was referenced Dec 29, 2017
Closed
Merged
@randallwhitman
Copy link
Contributor Author

randallwhitman commented Jan 2, 2018 •
edited
Loading

Should probably update Hive and Hadoop versions too.
[update: also Jackson version, to match Geometry v2.]

@randallwhitman randallwhitman changed the title Update Samples to Spatial Framework for Hadoop v2 and Geometry v2 Update Samples & Tutorials to Spatial Framework for Hadoop v2 and Geometry v2 Jan 4, 2018
@randallwhitman randallwhitman mentioned this issue Jan 6, 2018
Merged
@randallwhitman
Copy link
Contributor Author

Updated the tutorials today. @smambrose

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@randallwhitman randallwhitman

@smambrose smambrose

Labels
bug enhancement
Projects
None yet
Milestone
v2.1
Development

No branches or pull requests
2 participants
@randallwhitman @smambrose