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.

Sign up for GitHub

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

Jump to bottom

Update Bigtable branch with latest from master #1190

Merged
merged 21 commits into from
Jul 23, 2018
Merged

Update Bigtable branch with latest from master #1190

merged 21 commits into from
Jul 23, 2018

Conversation

dwsupplee
Copy link
Contributor

Includes a change set which resolves the CI error.

/cc @sduskis @ajaaym

RobLoach and others added 21 commits July 10, 2018 13:59
@RobLoach @jdpedrie
Add isRacy() to SafeSearch (googleapis#1166)
5b7fac0 
The SafeSearch class included `is` functions for each SafeSearch feature (adult, spoof, medical, violence), but was missing Racy. This Pull Request adds `isRacy()` to the SafeSearch class.
@jdpedrie
Exclude component vendor folder from snippet coverage (googleapis#1168)
8d95287 
cc @tmatsuo
@dwsupplee @jdpedrie
Allow context to handle Throwable interface (googleapis#1164)
84d4761
@dwsupplee
Add getServiceAccount to the BigQueryClient (googleapis#1167)
1234daf 
* Add getServiceAccount to the BigQueryClient

See:
https://cloud.google.com/bigquery/docs/reference/rest/v2/projects/getServiceAccount

* Added $options to getServiceAccount and fixed the indent

* Use single quote

* Add snippet test

* Comment update, use self instead of $this

* Comment update
@dwsupplee
Prepare v0.71.0 (googleapis#1169)
b47d699 
* Prepare v0.71.0

* patch release for Logging

* Correct version for Logging
@dwsupplee
[Kms] Regenerate with the new gapic config (googleapis#1165)
aa17757 
* Update for the new gapic configuration

* Docs update for the new and nicer namespace

* Removed the files with the old namespace

* Use the new namespace in the code sample

Add back the deprecated files

* Changed the wording for the deprecation warning
@jdpedrie @dwsupplee
Fix firestore queries (googleapis#1161)
8f3de64
@dwsupplee
Bump gax to 0.35 (googleapis#1170)
19072fe
@jdpedrie @dwsupplee
Add an interactive release builder. (googleapis#1160)
c846ba5 
* Add an interactive release builder.

* Create build directory if it doesn't exist
@dwsupplee
Add getServiceAccount method to StorageClient (googleapis#1173)
0991cfb
@dpebot @dwsupplee
Re-generate library using Tasks/synth.py (googleapis#1174)
2070cd4 
* Re-generate library using Tasks/synth.py

* Use new namespace in user visible area, tweak the deprecation wording
@dwsupplee
Add support for Numeric type (googleapis#1172)
3c5c274 
* Add support for Numeric type

* Added a cast in Numeric's constructor, added system tests

* Allow '123.' and '.123', update tests
@jdpedrie
[Breaking Change] Add support for Document Snapshots in Firestore Que…
0488b2d 
…ry Cursors (googleapis#1162)

cc @schmidt-sebastian

Extracted from googleapis#923 and updated to address pull request comments.

Breaking change is the standardization in Query of using `InvalidArgumentException`, replacing various throws of `BadMethodCallException`.

Closes googleapis#851.
@jdpedrie
Fix Storage Requesterpays system tests (googleapis#1180)
6eb5bad 
The requester pays system tests have been broken for some time. This change fixes them.
Bandaid for protobuf 4761 (googleapis#1176)
a9d8ff2 
* Temporary workaround for protobuf extension issue

protocolbuffers/protobuf#4761

* Bump gax to 0.36

* Configure comparators for the unit test

* Revert back to normal TestCase

* Install protobuf extension in the PHP 7.2 test runner

* Revert to TestCase
@dwsupplee
Prepare v0.72.0 (googleapis#1181)
42e0b9f
@dwsupplee
Added a document for the time filter on BigQueryClient->jobs() (googl…
c358e8c 
…eapis#1183)
Narrowed the time filter in the system test (googleapis#1185)
bc22ba1
@dpebot @dwsupplee
Re-generate library using BigQueryDataTransfer/synth.py (googleapis#1184
2e224af 
)

* Re-generate library using BigQueryDataTransfer/synth.py

* Tweak the wording on the deprecation warning

Also use the new namespace in the sample code
@dwsupplee
pin auth version until we have a way to silence warnings (googleapis#…
4a49232 
…1189)
@dwsupplee
Merge remote-tracking branch 'origin/master' into bigtable
e8d893d
@dwsupplee dwsupplee requested a review from jdpedrie July 23, 2018 18:48
@googlebot
Copy link

So there's good news and bad news.

👍 The good news is that everyone that needs to sign a CLA (the pull request submitter and all commit authors) have done so. Everything is all good there.

😕 The bad news is that it appears that one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that here in the pull request.

Note to project maintainer: This is a terminal state, meaning the cla/google commit status will not change from this state. It's up to you to confirm consent of the commit author(s) and merge this pull request when appropriate.

@googlebot googlebot added the cla: no This human has *not* signed the Contributor License Agreement. label Jul 23, 2018
@dwsupplee dwsupplee added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels Jul 23, 2018
@googlebot googlebot removed the cla: yes This human has signed the Contributor License Agreement. label Jul 23, 2018
@googlebot
Copy link

☹️ Sorry, but only Googlers may change the label cla: yes.

@sduskis sduskis added the cla: yes This human has signed the Contributor License Agreement. label Jul 23, 2018
@googlebot
Copy link

A Googler has manually verified that the CLAs look good.

(Googler, please make sure the reason for overriding the CLA status is clearly documented in these comments.)

@sduskis sduskis merged commit c2f4028 into googleapis:bigtable Jul 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdpedrie jdpedrie jdpedrie approved these changes

@chingor13 chingor13 Awaiting requested review from chingor13

@tmatsuo tmatsuo Awaiting requested review from tmatsuo

Assignees
No one assigned
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@dwsupplee @googlebot @jdpedrie @sduskis @RobLoach @dpebot