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

Cnde 2264 Added the Views for nrt srte codeset, code_val_general and condition dim #227

Merged
merged 7 commits into from
Mar 11, 2025

Conversation

rseshadri99
Copy link
Collaborator

Notes

Added the Views for nrt srte codeset, code_val_general and condition dim

JIRA

  • Related story: Cnde 2264

Checklist

Types of changes

What types of changes does this PR introduces?

  • Bugfix
  • New feature
  • Breaking change

Testing

  • Does this PR has >90% code coverage?
  • Is the screenshot attached for code coverage?
  • Does the gradle build pass in your local?
  • Is the gradle build logs attached?

rseshadri99 and others added 3 commits February 26, 2025 07:15
nrt_srte_<table> all sync'd to rdb_modern with change Capture.
update liquibase cahnge log for the nrt srte views codset, codevaluegeneral and condition
These views are generalized to fetch data from metadata tables from odse.
nbs_page, page_condition_mapping,nbs_ui_metadata and nbs_Rdb_metadata are to be added to Cahge capture. Then the new table nt_odse_<table> to be replaced in these views
Added alter to v_nrt_nbs_page
Removed duplicate changeset for codeset,code_val_general_condiotion_dim views
@rseshadri99
Copy link
Collaborator Author

Updated the view with create or alter
removed duplicate changeset

Added alter to all metadata views for dyn datamart
Copy link
Collaborator

@upasanapattnaik-eq upasanapattnaik-eq left a comment

Choose a reason for hiding this comment

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

This looks good to go on my end.
Is this the first PR that needs to be merged? The sprocs are referenced by the views and should be created before them. The liquibase numbers would reflect as such.

@rseshadri99 rseshadri99 merged commit 56e13d1 into main Mar 11, 2025
@sveselev sveselev deleted the CNDE-2264 branch March 15, 2025 06:07
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