-
Notifications
You must be signed in to change notification settings - Fork 2k
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
DLP: Added sample for k anonymity with entity ID and deidentify cloud storage #3364
Conversation
Here is the summary of changes. You are about to add 2 region tags.
This comment is generated by snippet-bot.
|
4b9bd20
to
0785ab9
Compare
5ea10ad
to
33fb3ae
Compare
33fb3ae
to
50e0d43
Compare
4137cbd
to
22029a2
Compare
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.
Please address the minor review comments.
@dinesh-crest please fix merge conflicts. @annadata this particular file has merge conflicts very frequently. maybe we can think about how we change this file to avoid these in the future, or split the test file. |
ca26e87
to
6456726
Compare
@dinesh-crest Whenever you have some time, could you resolve the file conflicts that are popping up? |
… storage Added unit test cases for same
c5fe760
to
ddc47c0
Compare
Added unit test cases for same
Description
References:- https://cloud.google.com/dlp/docs/deidentify-storage#code-example
https://cloud.google.com/dlp/docs/visualizing_re-id_risk.md?g=0&l=119#step_1_calculate_k-anonymity_on_the_dataset
Note: Before submitting a pull request, please open an issue for discussion if you are not associated with Google.
Checklist
npm test
(see Testing)npm run lint
(see Style)GoogleCloudPlatform/nodejs-docs-samples
. Not a fork.