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

[WIP]feat(structure-comparison):implemention for OdcDBStructureComparator #1322

Closed

Conversation

PeachThinking
Copy link
Collaborator

@PeachThinking PeachThinking commented Dec 28, 2023

What type of PR is this?

type-feature
module-structure-comparison

What this PR does / why we need it:

OdcDBStructureComparator :Implement DBStructureComparator interface based on db-browser.
If we can call DBCat SDK in the future, we only need to write another DBCat DBStructureComparator interface implementation class, and the service layer does not need to be aware of it.

The table's columns, indexes, constraints, and partition object's comparison results will be sub-objects of the table DBObjectComparisonResult,refer to DBObjectComparisonResult#getSubDBObjectComparisonResult().
For table object, DBObjectComparisonResult#getChangeScript() only only record the change script of table options.

Add support alter table charset and collate code in MySQLTableEditor#generateUpdateTableOptionDDL. Oracle do not support alter table charset and collate.

Which issue(s) this PR fixes:

Fixes #816

Special notes for your reviewer:

Additional documentation e.g., usage docs, etc.:


@PeachThinking PeachThinking added the type-feature The functionality to be implemented label Dec 28, 2023
@PeachThinking PeachThinking added this to the ODC 4.2.4 milestone Dec 28, 2023
@PeachThinking PeachThinking self-assigned this Dec 28, 2023
@PeachThinking PeachThinking changed the title feat(structure-comparison):implement DBStructureComparator feat(structure-comparison):implemention for OdcSchemaStructureComparator Dec 28, 2023
@PeachThinking PeachThinking force-pushed the feat/jingtian_424_support_schema_structure_sync branch from 754c6a1 to 078b5b4 Compare December 28, 2023 07:12
@PeachThinking PeachThinking changed the title feat(structure-comparison):implemention for OdcSchemaStructureComparator feat(structure-comparison):implemention for OdcDBStructureComparator Dec 29, 2023
@PeachThinking PeachThinking force-pushed the feat/jingtian_424_support_schema_structure_sync branch from 33a7dde to 1faab7a Compare December 29, 2023 03:55
@PeachThinking PeachThinking changed the title feat(structure-comparison):implemention for OdcDBStructureComparator [WIP]feat(structure-comparison):implemention for OdcDBStructureComparator Dec 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type-feature The functionality to be implemented
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

1 participant