Skip to content

update marts#447

Merged
Bala-Sakabattula merged 1 commit intorelease-engineering:mainfrom
Bala-Sakabattula:marts-changing
Mar 30, 2026
Merged

update marts#447
Bala-Sakabattula merged 1 commit intorelease-engineering:mainfrom
Bala-Sakabattula:marts-changing

Conversation

@Bala-Sakabattula
Copy link
Copy Markdown
Collaborator

updated the marts to use cloud marts instead of jira data centre marts...

@qodo-code-review
Copy link
Copy Markdown

Review Summary by Qodo

Migrate Snowflake queries to use Jira Cloud marts

✨ Enhancement

Grey Divider

Walkthroughs

Description
• Update Snowflake query to use CLOUD_MARTS instead of MARTS
• Replace three table references in downstream issue query
• Migrate from Jira Data Centre to Jira Cloud data sources
Diagram
flowchart LR
  old["JIRA_DB.MARTS tables"]
  new["JIRA_DB.CLOUD_MARTS tables"]
  query["Snowflake query in downstream_issue.py"]
  old -- "migrate to" --> new
  query -- "updated references" --> new
Loading

Grey Divider

File Changes

1. sync2jira/downstream_issue.py ⚙️ Configuration changes +3/-3

Replace MARTS with CLOUD_MARTS schema references

• Updated SNOWFLAKE_QUERY to reference CLOUD_MARTS schema instead of MARTS
• Changed JIRA_REMOTELINK table reference from JIRA_DB.MARTS to JIRA_DB.CLOUD_MARTS
• Changed JIRA_ISSUE table reference from JIRA_DB.MARTS to JIRA_DB.CLOUD_MARTS
• Changed JIRA_PROJECT table reference from JIRA_DB.MARTS to JIRA_DB.CLOUD_MARTS

sync2jira/downstream_issue.py


Grey Divider

Qodo Logo

@qodo-code-review
Copy link
Copy Markdown

qodo-code-review bot commented Mar 30, 2026

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Grey Divider

Great, no issues found!

Qodo reviewed your code and found no material issues that require review

Grey Divider

ⓘ The new review experience is currently in Beta. Learn more

Grey Divider

Qodo Logo

Copy link
Copy Markdown
Collaborator

@webbnh webbnh left a comment

Choose a reason for hiding this comment

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

I'm still vexed that we actually have to make this change, but, given that, it's OK by me.

@Bala-Sakabattula Bala-Sakabattula merged commit 6a790b4 into release-engineering:main Mar 30, 2026
6 checks passed
@Bala-Sakabattula Bala-Sakabattula deleted the marts-changing branch March 30, 2026 18:46
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