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.

Sign up for GitHub

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

Allow masking without primary keys #5575

Merged
merged 30 commits into from
Jan 10, 2025
+619 −538
Merged

Allow masking without primary keys #5575

Changes from 1 commit
Commits
Show all changes
30 commits
Select commit Hold shift + click to select a range
8bd2f74
Allow masking without primary keys
galvana Dec 9, 2024
cadcdb7
Updating tests
galvana Dec 9, 2024
de3ce24
Separating overlapping keys in update value map
galvana Dec 9, 2024
93974b8
Fixing data type
galvana Dec 9, 2024
f274ae0
Sorting update map keys
galvana Dec 9, 2024
357b6ec
Removing primary keys from sample and test datasets
galvana Dec 9, 2024
fc1aacc
Simplifying generate_update_stmt and fixing tests
galvana Dec 9, 2024
0e11551
More cleanup
galvana Dec 9, 2024
6a10d87
Misc fixes
galvana Dec 9, 2024
001b8ec
Renaming identity_or_reference_fields_paths to incoming_field_paths
galvana Dec 9, 2024
f31fd3a
Merge branch 'main' into LA-95-masking-without-primary-keys
galvana Dec 9, 2024
8816be7
Re-adding continue on error
galvana Dec 9, 2024
cae8493
Adding individual timeouts to tests
galvana Dec 10, 2024
bb7714a
Fixing datasets
galvana Dec 10, 2024
0d43401
Fixing some tests
galvana Dec 10, 2024
b13632b
Fixing MongoDB dataset
galvana Dec 10, 2024
b0ef57d
Re-adding primary key to mongo_test.customer_details
galvana Dec 10, 2024
77a5770
Splitting out query configs and tests
galvana Dec 10, 2024
5d26b2f
Splitting out tests
galvana Dec 10, 2024
fb7e566
Merge branch 'split-query-config-files' into LA-95-masking-without-pr…
galvana Dec 10, 2024
647586f
Reverting most of the removal of primary keys + misc files
galvana Dec 10, 2024
7600ab4
Removing primary key requirement for BigQuery erasures
galvana Dec 10, 2024
dd8a3ad
Setting requires_primary_keys for select connectors + updating tests
galvana Dec 11, 2024
3c3c63c
Revert setting requires_primary_keys to False for SaaS connectors
galvana Dec 11, 2024
74f6b22
Merge branch 'split-query-config-files' into LA-95-masking-without-pr…
galvana Dec 19, 2024
191910d
Reverting SaaS dataset changes
galvana Jan 6, 2025
a78cbec
Reverting SaaS dataset changes
galvana Jan 6, 2025
bf03464
Fixing tests
galvana Jan 6, 2025
8d552b0
Adding tests
galvana Jan 7, 2025
dec9b21
Removing continue-on-error
galvana Jan 9, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Loading
Loading status checks…
Removing continue-on-error
@galvana
galvana authored Jan 9, 2025

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
commit dec9b211e093ffa8a41ebba7728cc977cd0cba41
1 change: 0 additions & 1 deletion .github/workflows/backend_checks.yml
View file
Original file line number Diff line number Diff line change
@@ -256,7 +256,6 @@ jobs:

runs-on: ubuntu-latest
timeout-minutes: 45
continue-on-error: true
steps:
- name: Download container
uses: actions/download-artifact@v4