-
Notifications
You must be signed in to change notification settings - Fork 29
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
Pulls from S3 bucket rather than from AWS URL #35
Comments
I would assume that a nice pull request to improve this would be welcome. Two things to check though:
|
Hi there, As for the proper benchmarking, you are correct. For me, anecdotally, it is a good deal quicker. Did this today:
Indicating that, for this one simple case, the movement from the S3 bucket is a bit quicker (13 seconds vs 1 minute) As for that EC2 instance: I was merely hoping that would be an option, not that it would replace any other capabilities. That if one wanted to use S3 buckets rather than AWS HTTP sites as the location to get data from, that option would exist. Regards, |
To pull files directly from the S3 URI (s3://...), the backend would need to utilize |
Hello there folks,
Was making use of this opendata, to get that new 0.25 degree data. I noticed something that I would like investigate.
As is stands now, the tools I see, namely 'client.py', pulls data from a URL. For example, something like:
wget https://ecmwf-forecasts.s3.eu-central-1.amazonaws.com/20240227/12z/0p25/enfo/20240227120000-0h-enfo-ef.index
I believe that, from an amazon AWS EC2 instance, this would be a faster pull mechanism:
aws s3 cp --no-sign-request s3://ecmwf-forecasts/20240227/12z/0p25/enfo/20240227120000-0h-enfo-ef.index .
Those are command line steps, of course, Inside client.py and such, it would be different tools. My description above was simply to show the difference between a pull via HTTPS and AWS S3.
Any chance of adding capability to pull from the S3 bucket (and thus, the AWS 'backbone') into an AWS EC2 instance, rather than HTTP?
Regards,
Brian E.
The text was updated successfully, but these errors were encountered: