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 17934 poc on rulemap implementation #132

Open
wants to merge 17 commits into
base: crest-master
Choose a base branch
from

Conversation

prijendev
Copy link
Contributor

@prijendev prijendev commented Mar 2, 2022

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 mentioned this pull request Mar 16, 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.

2 participants