Skip to content

Conversation

rmarianski
Copy link
Member

This updates the rawr and metatile s3 paths. It:

  • places the hex path prefix first
  • removes the unused path and layer options

@rmarianski rmarianski requested review from iandees and zerebubuth July 12, 2018 18:54
Copy link
Member

@zerebubuth zerebubuth left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

@rmarianski
Copy link
Member Author

I added the ability to configure the s3 tile key format in 88330bb.

@rmarianski rmarianski merged commit 44b1aa6 into master Jul 20, 2018
@rmarianski rmarianski deleted the s3-path branch July 20, 2018 18:09
zerebubuth added a commit to zerebubuth/tapalcatl-py that referenced this pull request Aug 17, 2018
This allows us to match a code change made in tilezen/tilequeue#344, where the layer (and leading slash) are not longer part of the hashed string to make the S3 key.

Added tests to make sure we're conforming to the new way of calculating hashes and still backwards compatible with the old way.
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.

3 participants