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.

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

feat: support IN/NOT_IN/NOT_EQUAL operators #5198

Merged
merged 13 commits into from
Jun 13, 2022

Conversation

vishwarajanand
Copy link
Contributor

@vishwarajanand vishwarajanand commented Mar 30, 2022

Adds implementation to support IN/NOT_IN/NOT_EQUAL operators for datastore PHP SDK

Changes:

  1. Added operators IN, NOT_IN and NOT_EQUAL
  2. Added Unit tests for the above operators

Tests

UTs before adding tests:

➜  Datastore git:(main) ✗ 
➜  Datastore git:(main) ✗ XDEBUG_MODE=coverage  ../vendor/bin/phpunit --verbose -c phpunit.xml.dist tests/Unit/Query/QueryTest.php     
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.28 with Xdebug 3.1.2
Configuration: /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/phpunit.xml.dist

.................................                                 33 / 33 (100%)

Time: 47 ms, Memory: 4.00MB

OK (33 tests, 48 assertions)
➜  Datastore git:(main) ✗ 

UTs after adding tests:

➜  Datastore git:(main) ✗ XDEBUG_MODE=coverage  ../vendor/bin/phpunit --verbose -c phpunit.xml.dist tests/Unit/Query/QueryTest.php
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.28 with Xdebug 3.1.2
Configuration: /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/phpunit.xml.dist

....................................                              36 / 36 (100%)

Time: 40 ms, Memory: 4.00MB

OK (36 tests, 51 assertions)
➜  Datastore git:(main) ✗ 

Reviewer Help Needed:

  1. System test methods
  2. Should I do end-to-end tests?
  3. Are these snippet tests enough?
➜  Datastore git:(datastore_operators) vendor/bin/phpunit --verbose -c phpunit-snippets.xml.dist tests/Snippet/Query/QueryTest.php
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.28 with Xdebug 3.1.2
Configuration: /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/phpunit-snippets.xml.dist

..............                                                    14 / 14 (100%)

Time: 1.15 seconds, Memory: 12.00MB

OK (14 tests, 17 assertions)
NOTICE: 99 uncovered snippets!
0: Class example: Google\Cloud\Datastore\Transaction[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:56

1: Method example: Google\Cloud\Datastore\Transaction::insert[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:87

2: Method example: Google\Cloud\Datastore\Transaction::insertBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:116

3: Method example: Google\Cloud\Datastore\Transaction::update[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:156

4: Method example: Google\Cloud\Datastore\Transaction::updateBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:196

5: Method example: Google\Cloud\Datastore\Transaction::upsert[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:235

6: Method example: Google\Cloud\Datastore\Transaction::upsertBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:272

7: Method example: Google\Cloud\Datastore\Transaction::delete[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:300

8: Method example: Google\Cloud\Datastore\Transaction::deleteBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:326

9: Method example: Google\Cloud\Datastore\Transaction::commit[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Transaction.php:351

10: Method example: Google\Cloud\Datastore\Transaction::lookup[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:82

11: Method example: Google\Cloud\Datastore\Transaction::lookupBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:126

12: Method example: Google\Cloud\Datastore\Transaction::runQuery[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:155

13: Method example: Google\Cloud\Datastore\Transaction::rollback[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:172

14: Class example: Google\Cloud\Datastore\ReadOnlyTransaction[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/ReadOnlyTransaction.php:53

15: Class example: Google\Cloud\Datastore\ReadOnlyTransaction[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/ReadOnlyTransaction.php:53

16: Method example: Google\Cloud\Datastore\ReadOnlyTransaction::lookup[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:82

17: Method example: Google\Cloud\Datastore\ReadOnlyTransaction::lookupBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:126

18: Method example: Google\Cloud\Datastore\ReadOnlyTransaction::runQuery[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:155

19: Method example: Google\Cloud\Datastore\ReadOnlyTransaction::rollback[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/TransactionTrait.php:172

20: Class example: Google\Cloud\Datastore\Cursor[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Cursor.php:40

21: Class example: Google\Cloud\Datastore\DatastoreClient[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:77

22: Class example: Google\Cloud\Datastore\DatastoreClient[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:77

23: Class example: Google\Cloud\Datastore\DatastoreClient[2]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:77

24: Method example: Google\Cloud\Datastore\DatastoreClient::key[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:207

25: Method example: Google\Cloud\Datastore\DatastoreClient::key[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:207

26: Method example: Google\Cloud\Datastore\DatastoreClient::keys[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:255

27: Method example: Google\Cloud\Datastore\DatastoreClient::keys[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:255

28: Method example: Google\Cloud\Datastore\DatastoreClient::entity[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

29: Method example: Google\Cloud\Datastore\DatastoreClient::entity[array]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

30: Method example: Google\Cloud\Datastore\DatastoreClient::entity[object_accessor]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

31: Method example: Google\Cloud\Datastore\DatastoreClient::entity[incomplete]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

32: Method example: Google\Cloud\Datastore\DatastoreClient::entity[custom_class_interface]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

33: Method example: Google\Cloud\Datastore\DatastoreClient::entity[custom_class_extends]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

34: Method example: Google\Cloud\Datastore\DatastoreClient::entity[exclude_indexes]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:378

35: Method example: Google\Cloud\Datastore\DatastoreClient::geoPoint[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:399

36: Method example: Google\Cloud\Datastore\DatastoreClient::blob[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:420

37: Method example: Google\Cloud\Datastore\DatastoreClient::blob[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:420

38: Method example: Google\Cloud\Datastore\DatastoreClient::int64[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:437

39: Method example: Google\Cloud\Datastore\DatastoreClient::cursor[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:456

40: Method example: Google\Cloud\Datastore\DatastoreClient::allocateId[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:481

41: Method example: Google\Cloud\Datastore\DatastoreClient::allocateIds[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:511

42: Method example: Google\Cloud\Datastore\DatastoreClient::transaction[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:537

43: Method example: Google\Cloud\Datastore\DatastoreClient::readOnlyTransaction[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:571

44: Method example: Google\Cloud\Datastore\DatastoreClient::insert[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:608

45: Method example: Google\Cloud\Datastore\DatastoreClient::insertBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:639

46: Method example: Google\Cloud\Datastore\DatastoreClient::update[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:685

47: Method example: Google\Cloud\Datastore\DatastoreClient::updateBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:726

48: Method example: Google\Cloud\Datastore\DatastoreClient::upsert[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:772

49: Method example: Google\Cloud\Datastore\DatastoreClient::upsertBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:815

50: Method example: Google\Cloud\Datastore\DatastoreClient::delete[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:853

51: Method example: Google\Cloud\Datastore\DatastoreClient::deleteBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:888

52: Method example: Google\Cloud\Datastore\DatastoreClient::lookup[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:933

53: Method example: Google\Cloud\Datastore\DatastoreClient::lookupBatch[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:985

54: Method example: Google\Cloud\Datastore\DatastoreClient::query[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1004

55: Method example: Google\Cloud\Datastore\DatastoreClient::gqlQuery[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1079

56: Method example: Google\Cloud\Datastore\DatastoreClient::gqlQuery[bindings]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1079

57: Method example: Google\Cloud\Datastore\DatastoreClient::gqlQuery[pos_bindings]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1079

58: Method example: Google\Cloud\Datastore\DatastoreClient::gqlQuery[literals]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1079

59: Method example: Google\Cloud\Datastore\DatastoreClient::gqlQuery[cursor]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1079

60: Method example: Google\Cloud\Datastore\DatastoreClient::runQuery[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreClient.php:1114

61: Class example: Google\Cloud\Datastore\DatastoreSessionHandler[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreSessionHandler.php:115

62: Class example: Google\Cloud\Datastore\DatastoreSessionHandler[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/DatastoreSessionHandler.php:115

63: Class example: Google\Cloud\Datastore\Query\GqlQuery[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Query/GqlQuery.php:93

64: Class example: Google\Cloud\Datastore\Query\GqlQuery[bindings]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Query/GqlQuery.php:93

65: Class example: Google\Cloud\Datastore\Query\GqlQuery[pos_bindings]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Query/GqlQuery.php:93

66: Class example: Google\Cloud\Datastore\Query\GqlQuery[literals]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Query/GqlQuery.php:93

67: Class example: Google\Cloud\Datastore\Query\GqlQuery[cursor]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Query/GqlQuery.php:93

68: Class example: Google\Cloud\Datastore\GeoPoint[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/GeoPoint.php:36

69: Class example: Google\Cloud\Datastore\Blob[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Blob.php:42

70: Class example: Google\Cloud\Datastore\Blob[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Blob.php:42

71: Class example: Google\Cloud\Datastore\Entity[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Entity.php:106

72: Class example: Google\Cloud\Datastore\Entity[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Entity.php:106

73: Method example: Google\Cloud\Datastore\Entity::get[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityTrait.php:110

74: Method example: Google\Cloud\Datastore\Entity::getProperty[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityTrait.php:128

75: Method example: Google\Cloud\Datastore\Entity::set[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityTrait.php:151

76: Method example: Google\Cloud\Datastore\Entity::setProperty[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityTrait.php:171

77: Method example: Google\Cloud\Datastore\Entity::key[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityTrait.php:186

78: Method example: Google\Cloud\Datastore\Entity::cursor[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:49

79: Method example: Google\Cloud\Datastore\Entity::baseVersion[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:71

80: Method example: Google\Cloud\Datastore\Entity::populatedByService[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:88

81: Method example: Google\Cloud\Datastore\Entity::setExcludeFromIndexes[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:109

82: Method example: Google\Cloud\Datastore\Entity::excludedProperties[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:124

83: Method example: Google\Cloud\Datastore\Entity::meanings[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/EntityOptionsTrait.php:141

84: Class example: Google\Cloud\Datastore\Key[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:74

85: Class example: Google\Cloud\Datastore\Key[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:74

86: Class example: Google\Cloud\Datastore\Key[2]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:74

87: Method example: Google\Cloud\Datastore\Key::pathElement[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:166

88: Method example: Google\Cloud\Datastore\Key::pathElement[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:166

89: Method example: Google\Cloud\Datastore\Key::ancestor[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:215

90: Method example: Google\Cloud\Datastore\Key::ancestor[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:215

91: Method example: Google\Cloud\Datastore\Key::ancestorKey[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:243

92: Method example: Google\Cloud\Datastore\Key::state[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:284

93: Method example: Google\Cloud\Datastore\Key::state[1]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:284

94: Method example: Google\Cloud\Datastore\Key::setLastElementIdentifier[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:309

95: Method example: Google\Cloud\Datastore\Key::path[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:330

96: Method example: Google\Cloud\Datastore\Key::pathEnd[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:345

97: Method example: Google\Cloud\Datastore\Key::pathEndIdentifier[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:365

98: Method example: Google\Cloud\Datastore\Key::pathEndIdentifierType[0]
Declared on or around /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/src/Key.php:392

➜  Datastore git:(datastore_operators) 

@vishwarajanand vishwarajanand requested a review from a team as a code owner March 30, 2022 09:38
@vishwarajanand vishwarajanand self-assigned this Mar 30, 2022
@vishwarajanand vishwarajanand requested a review from a team as a code owner March 30, 2022 09:38
@vishwarajanand vishwarajanand added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 30, 2022
@vishwarajanand vishwarajanand changed the title [DO NOT MERGE] feat: support IN/NOT_IN/NOT_EQUAL operators feat: support IN/NOT_IN/NOT_EQUAL operators Mar 30, 2022
gcf-owl-bot bot and others added 6 commits April 4, 2022 06:55
…oogleapis#5115)

* fix!: removing misspelled field, add correctly spelled field

PiperOrigin-RevId: 428023165

Source-Link: googleapis/googleapis@63d374d

Source-Link: googleapis/googleapis-gen@6be9b6c
Copy-Tag: eyJwIjoiUHViU3ViLy5Pd2xCb3QueWFtbCIsImgiOiI2YmU5YjZjNmI2Yzg1ZmFjZTE4YWRiZGJkZDNmYzA2MzU2ODEwYjRhIn0=

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
…tps://google.aip.dev/123#annotating-resource-types (googleapis#5199)

* fix!: Updated resource patterns to comply with https://google.aip.dev/123#annotating-resource-types

PiperOrigin-RevId: 438230444

Source-Link: googleapis/googleapis@1821985

Source-Link: googleapis/googleapis-gen@30cb53f
Copy-Tag: eyJwIjoiQ2VydGlmaWNhdGVNYW5hZ2VyLy5Pd2xCb3QueWFtbCIsImgiOiIzMGNiNTNmOTlkMjUyYjM2NmFhODM3YjM4Y2Y5ZDFmOGIyODNlYzRhIn0=

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
…cking store used (googleapis#5162)

* feat: Added DatabaseType field for the type of backing store used
feat: Added AVRO DatabaseDumpSpec for importing and exporting Avro files
feat: Added additional endTime field for MetadataImports
feat: Added RESTORING status on Backups
feat: Added support to record the services that are restoring the backup
feat: Added configuration for Dataplex integration
feat: Added support for IAM management for metadata resources
feat: Added AuxiliaryVersionConfig for configuring the auxiliary hive versions during creation or update of the DPMS instance
feat: Added NetworkConfig for exposing the DPMS endpoint in multiple subnetworks using PSC (this skips the need for VPC peering)
feat: Added EncryptionConfig which contains information used to configure the Dataproc Metastore service to encrypt customer data at rest (CMEK)
docs: formatting improvements

PiperOrigin-RevId: 436271267

Source-Link: googleapis/googleapis@463caca

Source-Link: googleapis/googleapis-gen@e1cd573
Copy-Tag: eyJwIjoiRGF0YXByb2NNZXRhc3RvcmUvLk93bEJvdC55YW1sIiwiaCI6ImUxY2Q1NzMyMWM2YmM2ZWU1OTljMWNiM2I4MzE3OTQ1OTgwNDhhNmYifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
…to support fuzzy matching (googleapis#5159)

* docs: Added functionality in the companyDisplayNames filter to support fuzzy matching

PiperOrigin-RevId: 435467010

Source-Link: googleapis/googleapis@cb38285

Source-Link: googleapis/googleapis-gen@799e10e
Copy-Tag: eyJwIjoiVGFsZW50Ly5Pd2xCb3QueWFtbCIsImgiOiI3OTllMTBlODJkMDRhYTI3YWFmYmM2YzUzN2Y1YmUxZWFmODkxY2FkIn0=

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* feat: add WarmAndPing request for channel priming

PiperOrigin-RevId: 428795660

Source-Link: googleapis/googleapis@6cce671

Source-Link: googleapis/googleapis-gen@2282bc1
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjIyODJiYzFiMDgxMzY0ZWE3ODMzMDBiZTkxYThjMTRjYjRhNzE4YzQifQ==

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: Add ListHotTablets API method and protobufs

PiperOrigin-RevId: 436758628

Source-Link: googleapis/googleapis@92ab86a

Source-Link: googleapis/googleapis-gen@931ef11
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjkzMWVmMTE0ZDVmODQ1YWJmMTE3YmY4ZTBhMjk4MzZjYTMwMGI2OTQifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* docs: Update `cpu_utilization_percent` limit
docs: Remove the limitation of all clusters in a CMEK instance must use the same key

PiperOrigin-RevId: 438385300

Source-Link: googleapis/googleapis@c59f02e

Source-Link: googleapis/googleapis-gen@04d03d1
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjA0ZDAzZDE3YWFmYTdiNDQyMmY3M2M5MzYwMGYwNDA1NDI4MTdmY2QifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
…a3 protos. (googleapis#5137)

* fix!: Use http binding with location field as primary http bindings

Changing HTTP bindings and/or their order might be a breaking change for libraries.

PiperOrigin-RevId: 430239565

Source-Link: googleapis/googleapis@71fe7ff

Source-Link: googleapis/googleapis-gen@1f496cc
Copy-Tag: eyJwIjoiRGF0YWZsb3cvLk93bEJvdC55YW1sIiwiaCI6IjFmNDk2Y2MzNmRkNzc5MjFlNDVhMjc4MTE0YmJjMjE5OGNjODM1MmEifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: Add the ability to plumb environment capabilities through v1beta3 protos.

PiperOrigin-RevId: 434899727

Source-Link: googleapis/googleapis@392ae91

Source-Link: googleapis/googleapis-gen@febf173
Copy-Tag: eyJwIjoiRGF0YWZsb3cvLk93bEJvdC55YW1sIiwiaCI6ImZlYmYxNzMxNzI0ODNmY2ViZDU2NjhkZDMyZjNkNzM1MmE0NDIwY2UifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
@vishwarajanand vishwarajanand requested review from a team as code owners April 4, 2022 06:55
@vishwarajanand
Copy link
Contributor Author

tests/System/DatastoreSessionHandlerTest.php --> testSessionHandler test which fails here in CI but passes on my local and also seems somewhat unrelated to my changes. Parking this as of now and only to be reverted once we have a go ahead for the release.

➜  Datastore git:(datastore_operators) ✗ XDEBUG_MODE=develop vendor/bin/phpunit --verbose -c phpunit-system.xml.dist tests/System/DatastoreSessionHandlerTest.php --filter="testSessionHandler"
PHPUnit 5.7.27 by Sebastian Bergmann and contributors.

Runtime:       PHP 7.4.28 with Xdebug 3.1.2
Configuration: /usr/local/google/home/vishwarajanand/github/google-cloud-php/Datastore/phpunit-system.xml.dist

.                                                                   1 / 1 (100%)

Time: 1.32 seconds, Memory: 8.00MB

OK (1 test, 1 assertion)
➜  Datastore git:(datastore_operators) ✗

Copy link
Contributor

@bshaffer bshaffer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is LGTM from me but we should wait for @dwsupplee to review!

Copy link
Contributor

@dwsupplee dwsupplee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is looking nice! Would we be able to also add the not equals operator to the documented list of short operators on line 255 (the filter method)?

Datastore/src/Query/Query.php Outdated Show resolved Hide resolved
@dwsupplee
Copy link
Contributor

Friendly reminder about the request to add the new short operator to the documentation for the filter method (I'd comment on the line, but since there is no diff there github won't let me).

@vishwarajanand vishwarajanand removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 13, 2022
@dwsupplee dwsupplee merged commit 8bdc637 into googleapis:main Jun 13, 2022
vishwarajanand added a commit that referenced this pull request Jul 2, 2022
* Added new operators and unit tests

* fix: [PubSub] removing misspelled field, add correctly spelled field (#5115)

* fix!: removing misspelled field, add correctly spelled field

PiperOrigin-RevId: 428023165

Source-Link: googleapis/googleapis@63d374d

Source-Link: googleapis/googleapis-gen@6be9b6c
Copy-Tag: eyJwIjoiUHViU3ViLy5Pd2xCb3QueWFtbCIsImgiOiI2YmU5YjZjNmI2Yzg1ZmFjZTE4YWRiZGJkZDNmYzA2MzU2ODEwYjRhIn0=

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* fix: [CertificateManager] Updated resource patterns to comply with https://google.aip.dev/123#annotating-resource-types (#5199)

* fix!: Updated resource patterns to comply with https://google.aip.dev/123#annotating-resource-types

PiperOrigin-RevId: 438230444

Source-Link: googleapis/googleapis@1821985

Source-Link: googleapis/googleapis-gen@30cb53f
Copy-Tag: eyJwIjoiQ2VydGlmaWNhdGVNYW5hZ2VyLy5Pd2xCb3QueWFtbCIsImgiOiIzMGNiNTNmOTlkMjUyYjM2NmFhODM3YjM4Y2Y5ZDFmOGIyODNlYzRhIn0=

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: [DataprocMetastore] Added DatabaseType field for the type of backing store used (#5162)

* feat: Added DatabaseType field for the type of backing store used
feat: Added AVRO DatabaseDumpSpec for importing and exporting Avro files
feat: Added additional endTime field for MetadataImports
feat: Added RESTORING status on Backups
feat: Added support to record the services that are restoring the backup
feat: Added configuration for Dataplex integration
feat: Added support for IAM management for metadata resources
feat: Added AuxiliaryVersionConfig for configuring the auxiliary hive versions during creation or update of the DPMS instance
feat: Added NetworkConfig for exposing the DPMS endpoint in multiple subnetworks using PSC (this skips the need for VPC peering)
feat: Added EncryptionConfig which contains information used to configure the Dataproc Metastore service to encrypt customer data at rest (CMEK)
docs: formatting improvements

PiperOrigin-RevId: 436271267

Source-Link: googleapis/googleapis@463caca

Source-Link: googleapis/googleapis-gen@e1cd573
Copy-Tag: eyJwIjoiRGF0YXByb2NNZXRhc3RvcmUvLk93bEJvdC55YW1sIiwiaCI6ImUxY2Q1NzMyMWM2YmM2ZWU1OTljMWNiM2I4MzE3OTQ1OTgwNDhhNmYifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* docs: [Talent] Added functionality in the companyDisplayNames filter to support fuzzy matching (#5159)

* docs: Added functionality in the companyDisplayNames filter to support fuzzy matching

PiperOrigin-RevId: 435467010

Source-Link: googleapis/googleapis@cb38285

Source-Link: googleapis/googleapis-gen@799e10e
Copy-Tag: eyJwIjoiVGFsZW50Ly5Pd2xCb3QueWFtbCIsImgiOiI3OTllMTBlODJkMDRhYTI3YWFmYmM2YzUzN2Y1YmUxZWFmODkxY2FkIn0=

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: introduce hot tablet/ping and warm RPCs (#5119)

* feat: add WarmAndPing request for channel priming

PiperOrigin-RevId: 428795660

Source-Link: googleapis/googleapis@6cce671

Source-Link: googleapis/googleapis-gen@2282bc1
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjIyODJiYzFiMDgxMzY0ZWE3ODMzMDBiZTkxYThjMTRjYjRhNzE4YzQifQ==

* 🦉 Updates from OwlBot

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: Add ListHotTablets API method and protobufs

PiperOrigin-RevId: 436758628

Source-Link: googleapis/googleapis@92ab86a

Source-Link: googleapis/googleapis-gen@931ef11
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjkzMWVmMTE0ZDVmODQ1YWJmMTE3YmY4ZTBhMjk4MzZjYTMwMGI2OTQifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* docs: Update `cpu_utilization_percent` limit
docs: Remove the limitation of all clusters in a CMEK instance must use the same key

PiperOrigin-RevId: 438385300

Source-Link: googleapis/googleapis@c59f02e

Source-Link: googleapis/googleapis-gen@04d03d1
Copy-Tag: eyJwIjoiQmlndGFibGUvLk93bEJvdC55YW1sIiwiaCI6IjA0ZDAzZDE3YWFmYTdiNDQyMmY3M2M5MzYwMGYwNDA1NDI4MTdmY2QifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* feat: Add the ability to plumb environment capabilities through v1beta3 protos. (#5137)

* fix!: Use http binding with location field as primary http bindings

Changing HTTP bindings and/or their order might be a breaking change for libraries.

PiperOrigin-RevId: 430239565

Source-Link: googleapis/googleapis@71fe7ff

Source-Link: googleapis/googleapis-gen@1f496cc
Copy-Tag: eyJwIjoiRGF0YWZsb3cvLk93bEJvdC55YW1sIiwiaCI6IjFmNDk2Y2MzNmRkNzc5MjFlNDVhMjc4MTE0YmJjMjE5OGNjODM1MmEifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* feat: Add the ability to plumb environment capabilities through v1beta3 protos.

PiperOrigin-RevId: 434899727

Source-Link: googleapis/googleapis@392ae91

Source-Link: googleapis/googleapis-gen@febf173
Copy-Tag: eyJwIjoiRGF0YWZsb3cvLk93bEJvdC55YW1sIiwiaCI6ImZlYmYxNzMxNzI0ODNmY2ViZDU2NjhkZDMyZjNkNzM1MmE0NDIwY2UifQ==

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>

* limiting shorthand operators to not equals

* Added short operator to filter documentation

Co-authored-by: gcf-owl-bot[bot] <78513119+gcf-owl-bot[bot]@users.noreply.github.com>
Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants