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

Tdl 17961 poc on rule map implementation #142

Open
wants to merge 1 commit into
base: crest-work
Choose a base branch
from

Conversation

prijendev
Copy link

Description of change

  • Created RuleMap class which contain following below method,

    • fill_rule_map_object_by_catalog
    • apply_ruleset_on_schema
    • apply_ruleset_on_stream_name
    • apply_rules_to_original_field
    • apply_ruleset_on_api_response
  • Added extra method add_child_into_metadata to add nested fields into metadata.

  • The catalog contains the original-name field in metadata which contains the actual name field and the breadcrumb contains the standard name of the field.

  • During sync mode, apply_ruleset_on_api_response would be called which converts the original field value to standard name, and then the transform would be applied.

Manual QA steps

Risks

Rollback steps

  • revert this branch

@prijendev prijendev changed the base branch from master to crest-work March 16, 2022 10:03
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