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

Fix createIntegrationConnector and update ComponentIdReport #8017

Merged
merged 1 commit into from
Jan 31, 2024

Conversation

mandy-chessell
Copy link
Contributor

Description

The createIntegrationConnector had its own implementation of createConnection. This was not working when the connection was a virtual connection so I switch it to use the generic handler version. This had different problems which are now fixed. Similar problem with createGovernanceService. The result is that there is one method for creating connectors in the generic handlers and it seems to work.

The BasicFilesMonitor integration connector has cataloguing folders with the file resource connector rather than the folder resource coonnector. This cause subsequent survey action services on the folder asset to fail.

The survey folder survey action service what having trouble ceating an external profile log file. This is resolved by using the Apache Commons FileUtils that is able to create intervening folders as part of a file create.

The ComponentIdReport now contains the data connectors, XTDB connector and survey action connectors.

Related Issue(s)

None

Testing

  • Cataloguing files and runing surveys on the resulting assets.
  • Configuring integration connectors with different types of connection

Release Notes & Documentation

These are just bug fixes.

Additional notes

None

Signed-off-by: Mandy Chessell <mandy.e.chessell@gmail.com>
@mandy-chessell mandy-chessell merged commit 96c2624 into odpi:main Jan 31, 2024
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