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

pmtiles upload is unreliable for large files #89

Closed
bdon opened this issue Oct 5, 2023 · 5 comments
Closed

pmtiles upload is unreliable for large files #89

bdon opened this issue Oct 5, 2023 · 5 comments

Comments

@bdon
Copy link
Member

bdon commented Oct 5, 2023

Look into S3 retries

@gimse
Copy link
Contributor

gimse commented Dec 7, 2023

Why is "pmtiles upload" needed? Is it not possible just to use the aws cli?

@bdon
Copy link
Member Author

bdon commented Dec 15, 2023

It isn't hard to implement as it's part of gocloud and it means one can make complete use of pmtiles with a S3-compatible bucket, a set of credentials, and a single CLI tool. But maybe the overlap of pmtiles users and those with access to awscli or rclone is 100%.

Last I checked the AWS CLI requires an installer step and multiple files which is more complex than a single-file binary like go-pmtiles. Thoughts?

@jamesscottbrown
Copy link
Contributor

Personally I find pmtiles upload useful: the AWS CLI is a bit heavy to install, and rclone is more cumbersome to use. I thought rclone required first defining the S3 bucket that you want to use as a remote in a config file; IIUC, this is now longer true and you can now use the :backend:path/to/dir syntax, but it's still much harder to figure out the correct command for rclone than for pmtiles upload.

But maybe the overlap of pmtiles users and those with access to awscli or rclone is 100%.

I have access to both of these tools, but not necessarily from the machine or VM where I am generating the tiles. For example, I sometimes need to generate tiles containing data in a PostgreSQL DB that I can't connect to directly from my workstation because of firewall rules.

@lauriemann
Copy link

Just my .02, but I'd never used rclone until yesterday when I needed to copy up my .pmtiles files, but I found it pretty simple to use the docker version of rclone to upload to R2, so not a lot of value in spending time building yet another copy mechanism, time is better spent on other parts of the pmtiles project

bdon added a commit that referenced this issue Sep 15, 2024
* Simplify upload, reduce multipart upload part size [#89]
@bdon
Copy link
Member Author

bdon commented Sep 15, 2024

Should be simplified and improved in 1.22.

@bdon bdon closed this as completed Sep 15, 2024
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

No branches or pull requests

4 participants