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

DG-1530 Changing the wildcard pattern for indexsearch to be more efficient #3221

Closed
wants to merge 1,053 commits into from

Conversation

hr2904
Copy link
Collaborator

@hr2904 hr2904 commented Jun 10, 2024

Change description

Whenever creating/updating a policy, if policyResources contains an all domains pattern, these steps would happen:

  • Convert this old pattern to new pattern i.e default/domain/*super
  • Remove any specific domain access to be removed, i.e If the resources have a. All Domains b. Some Domain QN, -> that specific domain will be taken out of the list, as its redundant.

Changing the domain wildcard alias also fixed a bug where , old pattern was getting stakeholder entities as well as domain entities, where we are only supposed to get domain entities.

Type of change

  • Bug fix (fixes an issue)

Related issues

Fix #1
TestCases considered and tested on here

Checklists

Development

  • Lint rules pass locally
  • Application changes have been tested thoroughly
  • Automated tests covering modified code pass

Security

  • Security impact of change has been considered
  • Code follows company security practices and guidelines

Code review

  • Pull request has a descriptive title and context useful to a reviewer. Screenshots or screencasts are attached as necessary
  • "Ready for review" label attached and reviewers assigned
  • Changes have been reviewed by at least one other contributor
  • Pull request linked to task tracker where applicable

hr2904 and others added 30 commits May 7, 2024 18:14
feat: add endpoint to repair tag mappings fields
DG 1298 Form a QualifiedName for Domains and Products
DG-1235 Fix default titles bloating issue
[beta] Sync with master branch for lineage Graph API
Handle issue moving subDomain to make it Super Domain
DG 1298 Handle issue moving subDomain to make it Super Domain
…7-migrations2

# Conflicts:
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/DataDomainQNMigrationService.java
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/MigrationService.java
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/preprocessor/PreProcessorUtils.java
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/preprocessor/datamesh/AbstractDomainPreProcessor.java
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/preprocessor/datamesh/DataDomainPreProcessor.java
#	repository/src/main/java/org/apache/atlas/repository/store/graph/v2/preprocessor/datamesh/DataProductPreProcessor.java
#	webapp/src/main/java/org/apache/atlas/web/rest/MigrationREST.java
Fixed a missed check for dataProduct
DG-1235 Fix persona alias extraction issue
DG-1235 Stakeholder preprocessor
DG-1235 Skip initial auth checks
nikhilbonte21 and others added 23 commits June 10, 2024 19:02
DG-1533 check create/update sub-domain/products permissions
feature: Added support for "search_after" in elasticsearch queries
DG-1533 check create/update sub-domain/products permissions
feature: Added support for "search_after" in elasticsearch queries
DG-1432 Bootstrap relationship policies for MQ related Items
…sent as a resource in updating alias, it would be converted to default/domain/*/super.
…ic domains are removed for redundancy and if old pattern exists, convert it to new pattern.
@hr2904 hr2904 closed this Jun 17, 2024
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.