code review 14804043: all: new and updated APIs

all: new and updated APIs

[bradfitz, 2013-10-17 21:48:02]

Hello golang-dev@googlegroups.com,

I'd like you to review this change to
https://code.google.com/p/google-api-go-client/

[bradfitz, 2013-10-17 21:53:15]

*** Submitted as
https://code.google.com/p/google-api-go-client/source/detail?r=a5bd9c1fdc8c ***

all: new and updated APIs

R=golang-dev
CC=golang-dev
https://codereview.appspot.com/14804043

[bradfitz, 2013-10-17 21:54:18]

$ time hg submit 14804043
...
Uploading patch for drive/v2/drive-api.json
Uploading patch for drive/v2/drive-gen.go
Uploading patch for freebase/v1-sandbox/freebase-api.json
Thu Oct 17 14:51:30 2013 uploading patch for
freebase/v1-sandbox/freebase-api.json
Uploading patch for freebase/v1-sandbox/freebase-gen.go
Uploading patch for freebase/v1/freebase-api.json
Uploading patch for freebase/v1/freebase-gen.go
Uploading patch for freebase/v1sandbox/freebase-api.json
Uploading patch for freebase/v1sandbox/freebase-gen.go
Uploading patch for games/v1/games-api.json
Uploading patch for games/v1/games-gen.go
Uploading patch for gamesmanagement/v1management/gamesmanagement-api.json
Uploading patch for gamesmanagement/v1management/gamesmanagement-gen.go
Uploading patch for gan/v1beta1/gan-api.json
...

real 2m41.494s
user 0m3.956s
sys 0m0.352s


I love codereview.py, rietveld, Python, and hg.



On Thu, Oct 17, 2013 at 2:53 PM, <bradfitz@golang.org> wrote:

> *** Submitted as
> https://code.google.com/p/**google-api-go-client/source/**
>
detail?r=a5bd9c1fdc8c<https://code.google.com/p/google-api-go-client/source/detail?r=a5bd9c1fdc8c>
> ***
>
>
> all: new and updated APIs
>
> R=golang-dev
> CC=golang-dev
>
https://codereview.appspot.**com/14804043<https://codereview.appspot.com/1480...
>
>
>
https://codereview.appspot.**com/14804043/<https://codereview.appspot.com/148...
>

[rsc, 2013-10-18 20:42:20]

codereview.appspot.com should be changed to allow bulk uploads (say, a zip
of the files), which would result in a single datastore op instead of many,
and codereview.py should be changed to use the new API. Neither has
happened.