-
Notifications
You must be signed in to change notification settings - Fork 697
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
[DOCS] add glue tutorial #1453
[DOCS] add glue tutorial #1453
Conversation
docs/setup/glue.md
Outdated
[Wherobots](https://wherobots.com/) provides a public S3 bucket with the necessary jars. You can point to these directly in your glue jobs' | ||
configurations. For 1.6.0, the sedona and geotools jars are available at the following locations: | ||
|
||
* `s3://wherobots-sedona-jars/1.6.0/sedona-spark-shaded-3.0_2.12-1.6.0.jar` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you use the variable {{ sedona.current_version }} instead of hard coded version. See the example in other tutorial.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
how can we ensure that these are updated when a new version of sedona is released?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Because I will update the variable name after each release: https://github.com/apache/sedona/blob/master/mkdocs.yml#L161
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, I mean the jars in the s3 bucket.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sadly, someone needs to add it manually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is there a release checklist or something to which we can add to ping me to update the bucket?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Checklist is here: https://sedona.apache.org/1.6.0/community/publish/
* add glue tutorial * revise glue tutorial with PR feedback * add spark/scala warning for picking out the jars --------- Co-authored-by: jameswillis <james@wherobots.com>
Did you read the Contributor Guide?
Is this PR related to a JIRA ticket?
[DOCS] my subject
.I am addressing this github issue: #1399
What changes were proposed in this PR?
This PR proposes adding an AWS Glue setup tutorial to the Sedona documentation website
How was this patch tested?
built and ran the docs site locally.

followed the tutorial using the wherobots jars in an account other than the one hosting the jars bucket.
Did this PR include necessary documentation updates?