-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Add samples for Data Catalog lookupEntry #1416
Add samples for Data Catalog lookupEntry #1416
Conversation
PTAL the new sample format in CONTRIBUTING.md @kurtisvg for additional info. |
It's highly suggested to use the updated sample format, which has guidelines here. This is to avoid having to update the samples again in the new future. |
My quick glance was that I didn't see the comments on the parameters being passed, so I didn't scan the rest of the sample. https://github.com/GoogleCloudPlatform/java-docs-samples/blob/master/SAMPLE_FORMAT.md And take a look here: https://github.com/GoogleCloudPlatform/java-docs-samples/tree/master/dlp/src/main/java/dlp/snippets |
Just pushed a new version with several changes. I believe it's more consistent with the recommended standards now. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PTAL at CI tests as well.
datacatalog/cloud-client/src/main/java/com/example/datacatalog/LookupEntryBigQueryDataset.java
Show resolved
Hide resolved
datacatalog/cloud-client/src/main/java/com/example/datacatalog/LookupEntryBigQueryDataset.java
Outdated
Show resolved
Hide resolved
datacatalog/cloud-client/src/main/java/com/example/datacatalog/LookupEntryBigQueryDataset.java
Outdated
Show resolved
Hide resolved
datacatalog/cloud-client/src/main/java/com/example/datacatalog/LookupEntryBigQueryDataset.java
Outdated
Show resolved
Hide resolved
...talog/cloud-client/src/test/java/com/example/datacatalog/LookupEntryBigQueryDatasetTest.java
Outdated
Show resolved
Hide resolved
Co-Authored-By: Kurtis Van Gent <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two nits, otherwise LGTM when the tests pass.
datacatalog/cloud-client/src/main/java/com/example/datacatalog/QuickStart.java
Outdated
Show resolved
Hide resolved
fixes #1410 |
Adding samples to Data Catalog lookup_entry, according to #1410