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

CJ-181 : Application Asset Creator and Tests (new typedef) #1008

Merged
merged 30 commits into from
Nov 15, 2024
Merged

Conversation

prateekrai-atlan
Copy link
Collaborator

⚠️

  • Generate module : step not complete, do not see addition of applicationAssetQualifiedName attribute at a catalog level for all assets under catalog.
  • Generate Test : step not complete, ApplicationAssetTest is empty right now.
  • Changed the already present Application Connector Type to go from SAAS to APPLICATION. (had 0 use earlier, done to keep both java and python sdks same in connector type classes)

🆕

New Asset Typedef called Application with one asset called Application Asset. Details and Need :

  • Application has a shared attribute called applicationId which is a string type and stores the ID of the application in the Application Asset.
  • Application Asset has a ONE-TO-MANY relationship with any asset under Catalog. Thus any asset under Catalog can have only one application asset as parent, but application asset has many child catalog assets.
  • A de-normalized attribute called applicationAssetQualifiedName to store the qualified name of the parent application asset at the catalog level. All catalog assets will inherit this attribute.

@prateekrai-atlan prateekrai-atlan added the help wanted Extra attention is needed label Nov 13, 2024
prateekrai-atlan and others added 25 commits November 13, 2024 22:52
Signed-off-by: Christopher Grote <[email protected]>
Reworked tests to match latest tenant setups
DVX-715 Change AZURE_STORAGE_CONTAINER_NAME to objectstore
Asset import improvements
Concurrency fixes for tests and off-heap caching
Attempt to resolve OOM in GitHub Action
Signed-off-by: Christopher Grote <[email protected]>
@cmgrote cmgrote requested a review from ErnestoLoma as a code owner November 15, 2024 22:10
Signed-off-by: Christopher Grote <[email protected]>
@cmgrote cmgrote merged commit 9ffb1f4 into main Nov 15, 2024
4 checks passed
@cmgrote cmgrote deleted the CJ-181 branch January 20, 2025 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants