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

Add Anchors classifications to archive elements #8038

Merged
merged 6 commits into from
Feb 6, 2024

Conversation

mandy-chessell
Copy link
Contributor

Description

This PR imp[roves the contents of the archives so that the elements have anchors classificaitons in them. This improves the performance of the repository just after the archive is loaded.

Related Issue(s)

  • There was a problem in the InvalidParameterHandler when a metadata access server had maxPageSize set to 0
  • The parameters for findGlossaryTerms and findGlossaryCategories in OMVS were being passed in the wrong order to Asset Manager OMAS. There still seems to be a prolbem in the RegEx masking used by these requests.
  • Git also complained about the size of the big glossary archives so they are now smaller - and more of them.

Testing

Manual testing of the archives.

Release Notes & Documentation

Just bug fixes here.

Additional notes

Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
@mandy-chessell mandy-chessell merged commit a6a2e04 into odpi:main Feb 6, 2024
4 checks passed

String assetDiscoveryEngineGUID = this.getAssetDiscoveryEngine();

Check notice

Code scanning / CodeQL

Unread local variable Note

Variable 'String assetDiscoveryEngineGUID' is never read.
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