-
Notifications
You must be signed in to change notification settings - Fork 1
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: add policy tags and masking policy support for maxcompute #78
base: main
Are you sure you want to change the base?
Conversation
0f32b34
to
b06064e
Compare
6a95ac6
to
e7d694c
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.
PR Overview
This PR adds support for masking policies and policy tags in maxcompute, enhancing both the client functionality and corresponding mocks used in testing.
- Added GetMaskingPolicy support in the maxcompute client and its corresponding mocks.
- Updated extractor functions to integrate the new masking policy.
- Upgraded artifact upload actions in the CI workflow.
Reviewed Changes
File | Description |
---|---|
plugins/extractors/maxcompute/mocks/maxcompute_client_mock.go | Added mock methods for GetMaskingPolicy to support testing needs. |
plugins/extractors/maxcompute/maxcompute.go | Updated extractor to pass the required table parameter and integrate masking policy. |
plugins/extractors/maxcompute/client/client.go | Implemented GetMaskingPolicy to retrieve masking policies from a table. |
.github/workflows/test.yml | Updated artifact upload actions from v3 to v4. |
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
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.
some small comments.
Btw have you test this locally to project that has masking policy?
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.
most are okay, but need to recheck the test as suggested by others
477bb8e
to
abf270b
Compare
abf270b
to
3c4e5c0
Compare
No description provided.