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: add policy tags and masking policy support for maxcompute #78

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

Mayurjag
Copy link

No description provided.

@Mayurjag Mayurjag force-pushed the feat/maxcompute-policy-tags-masking-policy branch from 0f32b34 to b06064e Compare January 23, 2025 04:06
@Mayurjag Mayurjag force-pushed the feat/maxcompute-policy-tags-masking-policy branch 4 times, most recently from 6a95ac6 to e7d694c Compare March 4, 2025 12:52
@Mayurjag Mayurjag marked this pull request as ready for review March 4, 2025 12:59
@haveiss haveiss requested a review from Copilot March 4, 2025 13:04
Copy link

@Copilot Copilot AI left a 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.

Copy link

@luthfifahlevi luthfifahlevi left a 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?

Copy link

@irainia irainia left a 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

@Mayurjag Mayurjag force-pushed the feat/maxcompute-policy-tags-masking-policy branch from 477bb8e to abf270b Compare March 12, 2025 11:38
@Mayurjag Mayurjag force-pushed the feat/maxcompute-policy-tags-masking-policy branch from abf270b to 3c4e5c0 Compare March 19, 2025 07:20
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.

5 participants