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

Make CatalogTarget relationship multi-link #8255

Merged
merged 2 commits into from
Jun 27, 2024

Conversation

mandy-chessell
Copy link
Contributor

Description

For the Unity Catalog integration connectors, we need to establish multiple CatalogTarget relationships between the Unity Catalog Server entity and the OSSUnityCatalogInsideCatalogSync integration connector entity (one for each catalog).

This means the CatalogTarget type definition needs to be set to multi-link and the REST API for managing catalog target relationships has to change to:

  • allow the relationship GUID to be passed on the getCatalogTarget, updateCatalogTarget and deleteCatalogTarget methods
  • return the relationshipGUID on the createCatalogTarget

These API changes affect:

  • Automated Curation OMVS
  • Governance Engine OMAS
  • Open Integration Framework Services

@fyi @dwolfson

Related Issue(s)

  • The release of 5.0 libraries to maven central identified a build.gradle file (GUIDGenerator) that did not have a description. This has been added.

  • Templates for the different types of files were using a placeholder variable for the deployedImplementationType attribute which has the follow problems:

    • The calling user has to set it - annoying - and they may get it wrong.
    • The method in Automated Curation OMVS that allows templates to be queried by technology type (ie deployedImplementationType) was not finding the file templates.

This is now fixed in the CoreContentpack.

The GitHub workflows for merge and release are no longer publishing the egeria-configure container as we no longer need it.

Testing

Lots of manual testing through the OSS Unity Catalog connectors.

Release Notes & Documentation

The change to the CatalogTarget relationship is now inthe release notes.

Additional notes

None

@mandy-chessell mandy-chessell requested a review from planetf1 as a code owner June 27, 2024 15:18
@mandy-chessell mandy-chessell enabled auto-merge June 27, 2024 15:18
@mandy-chessell mandy-chessell merged commit 1565faf into odpi:main Jun 27, 2024
3 of 4 checks passed
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.

1 participant