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

DE Proxy - use ProcessingState classification for saving sync time in DS connector #7009

Merged
merged 13 commits into from
Nov 9, 2022

Conversation

alexandra-bucur
Copy link

Description

Until now the sync state of the Data Stage Proxy was saved in the IGC rules. These changes move the sync state into the ProcessingState classification that is related to an Engine entity.
Also, this PR includes the remaining pieces for the refactoring from SoftwareServerCapability to Engine.

Testing

Manually with IGC, Data Stage Proxy, IBM Proxy, Data Engine, UI + JUnits + FVTs

Alexandra Bucur added 5 commits October 19, 2022 10:12
… DS connector

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
…ing state classification

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
@sonatype-lift
Copy link
Contributor

sonatype-lift bot commented Oct 19, 2022

⚠️ 110 God Classes were detected by Lift in this project. Visit the Lift web console for more details.

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
Alexandra Bucur added 3 commits October 20, 2022 19:05
Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
…nector v1 (no reports)

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
Copy link
Contributor

@lpalashevski lpalashevski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve with 2 minor comments coming from sonatype lift scan.

Signed-off-by: Alexandra Bucur <alexandra-nona.bucur@ing.com>
@alexandra-bucur alexandra-bucur merged commit d9f0262 into odpi:main Nov 9, 2022
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.

3 participants