{"push_id":2972476867,"size":201,"distinct_size":201,"ref":"refs/heads/master","head":"1d9633a229077f680bf8fe431a93d5f489835506","before":"a55dfc0b29d5e9df654f451579bb012d9417b14b","commits":[{"sha":"a267b3d005e3a0f5886bf298bd27330332cb55a3","author":{"name":"Jean de Klerk","email":"605b93664484fed1e671a07bff68da39e73d0174@google.com"},"message":"kms: regen gapic; removes alpha notice\n\nChange-Id: I38dbe356058e28c4effdfc910229e72db1adc586\nReviewed-on: https://code-review.googlesource.com/30590\nReviewed-by: Jonathan Amsterdam <jba@google.com>\nReviewed-by: kokoro <noreply+kokoro@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/a267b3d005e3a0f5886bf298bd27330332cb55a3"},{"sha":"f4024bfe9905677af1b3705dc5299b82622933c7","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"pubsub: support labels and geofencing\n\nThis CL supports topic MessageStoragePolicy (aka \"geofencing\") as well\nas labels on topics and subscriptions.\n\n- Subscription: Create with, get and update labels.\n- Topic:\n  - Get and update labels (but not create, because it would be a breaking change).\n  - Get and \"update\" MessageStoragePolicy.\n\nMessageStoragePolicy is unusual. You can't set or update it because\nit's configured at the project or organization level. In order to observe\nit in the topic resource, you have to call UpdateTopic with\n\"message_storage_policy\" in the field mask (but, again, you must *not*\npopulate the actual field).\n\nThe CL is missing an integration test for MessageStoragePolicy,\nbecause that requires configuring a new project in a special testing\norg. A CL with the test will be forthcoming.\n\nChange-Id: I0c72cc4c71520a299695acc21bfd3d477bba9eba\nReviewed-on: https://code-review.googlesource.com/30550\nReviewed-by: Jean de Klerk <deklerk@google.com>\nReviewed-by: kokoro <noreply+kokoro@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/f4024bfe9905677af1b3705dc5299b82622933c7"},{"sha":"ffb8ba1de408dae97a525365cb711bd2fa28f8ef","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"bigquery: don't call tabledata.list if there are no rows in the query result\n\nIf jobs.getQueryResults says that the total rows in the result is\nzero, have RowIterator.Next return immediately with iterator.Done,\nwithout calling tabledata.list.\n\nThis allows queries with DML statements to be treated just like any\nother query: callers can attempt to iterate over the resulting\nrows. (They will get iterator.Done on calls to RowIterator.Next.)\nWithout this change, the call to tabledata.list will fail.\n\nChange-Id: I9c051449ecc6307efbe20e9edc01ce6a7826e297\nReviewed-on: https://code-review.googlesource.com/30610\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/ffb8ba1de408dae97a525365cb711bd2fa28f8ef"},{"sha":"7b18bf67abfec72beb23fc10741b46f6f9d61a9f","author":{"name":"Seth Hollyman","email":"10194de3bbef2536f4b5d02c780e8788d90cab2e@google.com"},"message":"bigquery: include job creator email in Job representation.\n\nChange-Id: Ic4e7dd15ec8bff40739ed11630000e1322c21cff\nReviewed-on: https://code-review.googlesource.com/30630\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jonathan Amsterdam <jba@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/7b18bf67abfec72beb23fc10741b46f6f9d61a9f"},{"sha":"1bad0581757551231811dfc864d6d5760deabe48","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"pubsub: add a test for message storage policy\n\nChange-Id: I786f171e3feaaf2cb475c0a212bc29aee668e220\nReviewed-on: https://code-review.googlesource.com/30670\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/1bad0581757551231811dfc864d6d5760deabe48"},{"sha":"15ebef8a76b24297d4e908e81af5df1569f5ccaf","author":{"name":"Alexey Alexandrov","email":"707c85aa1ca9cc766e4957d3fb95a91b4e895454@google.com"},"message":"Remove the mmap len check, as github.com/google/pprof/issues/242 is fixed.\n\nChange-Id: I2bb07b748d1b683ba7337194b21e428678531f8d\nReviewed-on: https://code-review.googlesource.com/30690\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jonathan Amsterdam <jba@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/15ebef8a76b24297d4e908e81af5df1569f5ccaf"},{"sha":"bd825f5c35f42dc10a150a6b42284e70dce4536e","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"pubsub: fix broken test\n\nIn the previous CL, I missed that after being whitelisted, the test\nproject now has a set of resource location restrictions.\n\nChange-Id: Ia408c7e11c79f9e82f1f43fa77f778f78387eaba\nReviewed-on: https://code-review.googlesource.com/30710\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/bd825f5c35f42dc10a150a6b42284e70dce4536e"},{"sha":"2decb91d506b3662e43a07560973131b9da58f0e","author":{"name":"Alexey Alexandrov","email":"707c85aa1ca9cc766e4957d3fb95a91b4e895454@google.com"},"message":"Update the tests to compile / pass with the recent API updates.\n\nAdded instructions on the manual steps to regenerate mocks, this\nwasn't documented so I had to reconstruct the steps to take.\n\nChange-Id: I5b6a133ac73801453341910a129f2e8d181cf58f\nReviewed-on: https://code-review.googlesource.com/30730\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jonathan Amsterdam <jba@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/2decb91d506b3662e43a07560973131b9da58f0e"},{"sha":"122c810c72569ea143e5d54ed2b041f4b3e56200","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: support bucket logging\n\nFixes #1041.\n\nChange-Id: I73a6149cade6fe4b68125ea75e4ce3874d1da299\nReviewed-on: https://code-review.googlesource.com/30711\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/122c810c72569ea143e5d54ed2b041f4b3e56200"},{"sha":"37d75fbba39c9b5b5df94a8ffa335c4d271118f3","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: clean up the transport used for testing\n\n- Move fakeTransport to its own file.\n- Rename it mockTransport, because it's really a mock.\n- Provide a mockClient function to simplify use.\n\nChange-Id: I679696f9263cf7421884863b8f89cb19c4385e61\nReviewed-on: https://code-review.googlesource.com/30713\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/37d75fbba39c9b5b5df94a8ffa335c4d271118f3"},{"sha":"cd5cf31db191bcec9487d6d333de6cf12bffe460","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: support additional ACL fields\n\nAdd support for additional fields of bucket and object ACLs.\n\nThese fields can only be read, not written.\n\nFixes #1043.\n\nChange-Id: I7022e5e064af92a40cf15577a2a02244c56e5bd3\nReviewed-on: https://code-review.googlesource.com/30770\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/cd5cf31db191bcec9487d6d333de6cf12bffe460"},{"sha":"5a102bbe5f5c736496df920decb0e9f579d4edf2","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"profiler: fix bad storage.ACLRule literal\n\nChange-Id: Id293053b44f68ed4501bcde8a656b61f510a0047\nReviewed-on: https://code-review.googlesource.com/30771\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jonathan Amsterdam <jba@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/5a102bbe5f5c736496df920decb0e9f579d4edf2"},{"sha":"f05ab167efe4a7a9b334f85ed0195500a5847150","author":{"name":"alokc","email":"298ff75bf36f74f3109ce385a418abdd09296991@gmail.com"},"message":"added timerange range for delete column families\n\nupdated test cases and added validation for end timestamp\nfixes #932\n\nChange-Id: Ia667872883662cd45f2ac36f7e13cf3a5976783c\nReviewed-on: https://code-review.googlesource.com/29430\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jonathan Amsterdam <jba@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/f05ab167efe4a7a9b334f85ed0195500a5847150"},{"sha":"56cca5e5698466b058b205f1b251a42ff5641815","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: support bucket websites\n\nFixes #1042.\n\nChange-Id: I9aa27e6588a583a439120f58e477ad7d60133dc1\nReviewed-on: https://code-review.googlesource.com/30712\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/56cca5e5698466b058b205f1b251a42ff5641815"},{"sha":"e615d33101b4ccf8ef205a981ac768c3a6a78d61","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: support predefined ACLs\n\nFixes #61.\n\nChange-Id: Id082c374e6f74bfb0766485fb116c2deb7253b78\nReviewed-on: https://code-review.googlesource.com/30810\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/e615d33101b4ccf8ef205a981ac768c3a6a78d61"},{"sha":"3030226c3a428c562daaadad813265042e4bba04","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"firestore: clarify doc for Stop\n\nMake it clear that you don't have to call Stop on every iterator,\njust those with a Stop method.\n\nFixes #1075.\n\nChange-Id: I5afed1bf3e9fc520d15f0abc16d5d4df9976d3f4\nReviewed-on: https://code-review.googlesource.com/30830\nReviewed-by: Daniel Martí <mvdan@mvdan.cc>\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/3030226c3a428c562daaadad813265042e4bba04"},{"sha":"fd07ebf75dcf9a34c69871fa0e27ac67d8cdd08c","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"datastore: save non-nil pointer fields on omitempty\n\nIf a pointer field is marked omitempty and contains a non-nil pointer,\nthen it should be saved. That's true even if the non-nil pointer\nrefers to a zero value.\n\nE.g.\n\n  type S struct { F *int `datstore:\"f,omitempty\"` }\n  zero := 0\n  s := &S{F: &zero}\n\nBefore this CL, the F field was not written to the database. It\nshould be, however, because it is not empty -- it is not a nil\npointer. With this CL, the field is written.\n\nNow this case matches the behavior of encoding/json.\n\nFixes #920.\n\nChange-Id: Iaadae511bbb76b15bc627c53e9f5a25f61c23c41\nReviewed-on: https://code-review.googlesource.com/30831\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/fd07ebf75dcf9a34c69871fa0e27ac67d8cdd08c"},{"sha":"da4f335b11aa8c8b46ddf6073ae54358cbdbe6b9","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"storage: document how to use contexts to stop retrying forever\n\nFixes #940.\n\nChange-Id: I542bf71c96a226be77793bb69957c7e9e2eac2eb\nReviewed-on: https://code-review.googlesource.com/30850\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/da4f335b11aa8c8b46ddf6073ae54358cbdbe6b9"},{"sha":"6cd26c9b628d44ad33eab21b8178b2cea33acf25","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"datastore: document that time fields are retrieved as local time.\n\nFixes #916.\n\nChange-Id: I04f2f6eef768ab72e052283947fe7877b9f728e6\nReviewed-on: https://code-review.googlesource.com/30892\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/6cd26c9b628d44ad33eab21b8178b2cea33acf25"},{"sha":"d4056dd406ac25758d3ec07f6b48c51d6188d73a","author":{"name":"Jonathan Amsterdam","email":"e3d3698b2ccd5955e4adf250d0785062d0f9018b@google.com"},"message":"firestore: remove ErrConcurrentTransaction\n\nIt was never actually used anywhere.\n\nFixed doc as well.\n\nChange-Id: Ic248b8cc723f01f5c95eaa1d7bb13367ed77df79\nReviewed-on: https://code-review.googlesource.com/30891\nReviewed-by: kokoro <noreply+kokoro@google.com>\nReviewed-by: Jean de Klerk <deklerk@google.com>","distinct":true,"url":"https://api.github.com/repos/commonnet/gcloud-golang/commits/d4056dd406ac25758d3ec07f6b48c51d6188d73a"}]}
{"actor":{"display_login":"amrik"}}