Increase maximum file upload to 100MB, use cloudstorage python library

Increase maximum file upload to 100MB, use cloudstorage python library

Using the deprecated files API from AppEngine can potentially result in errors
like "FileNotOpenedError: ApplicationError: 10" -- this seems to be caused by
a 30 second timeout.
(See deprecation notice here:
 https://cloud.google.com/appengine/docs/python/googlestorage/functions)

Using the replacement "cloudstorage" is either not suffering from this problem
or is just faster
(See:
 https://cloud.google.com/appengine/docs/python/googlecloudstorageclient/)

R=rnystrom@google.com, sgjesse@google.com

Committed: https://github.com/dart-lang/pub-dartlang/commit/854605603f17224dc40ac3d8f196be96053e3dd4

[kustermann, 2015-03-23 14:47:23]

kustermann@google.com changed reviewers:
+ rnystrom@google.com, sgjesse@google.com

[Bob Nystrom, 2015-03-23 19:41:13]

LGTM (though my memory of the pub server codebase is pretty weak these days, so
I may certainly have missed some stuff)!

https://chromiumcodereview.appspot.com/1031663002/diff/20001/app/handlers/clo...
File app/handlers/cloud_storage.py (right):

https://chromiumcodereview.appspot.com/1031663002/diff/20001/app/handlers/clo...
app/handlers/cloud_storage.py:218: def open(obj):
Might be good to leave a comment that this is deprecated, or just remove it
outright if it's not being used anymore.

[Søren Gjesse, 2015-03-24 12:16:40]

LGTM

[kustermann, 2015-03-24 18:21:55]

Patchset #3 (id:40001) has been deleted

[kustermann, 2015-03-24 18:28:14]

Committed patchset #3 (id:60001) manually as
854605603f17224dc40ac3d8f196be96053e3dd4 (presubmit successful).

[kustermann, 2015-03-24 18:28:21]

Thanks for the review.

https://chromiumcodereview.appspot.com/1031663002/diff/20001/app/handlers/clo...
File app/handlers/cloud_storage.py (right):

https://chromiumcodereview.appspot.com/1031663002/diff/20001/app/handlers/clo...
app/handlers/cloud_storage.py:218: def open(obj):
On 2015/03/23 19:41:13, Bob Nystrom wrote:
> Might be good to leave a comment that this is deprecated, or just remove it
> outright if it's not being used anymore.

Done.

It's still used -- I've added a deprecation comment.