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

Dev 11608 update clean postgres sql for spark #4231

Draft
wants to merge 5 commits into
base: qat
Choose a base branch
from

Conversation

zachflanders-frb
Copy link
Contributor

@zachflanders-frb zachflanders-frb commented Nov 25, 2024

Description:
This addresses issues with the clean_postgres_sql_for_spark_sql to enable the regex to work for replacing functions.

Technical details:
The updated regex will find matches including between ( ( characters and then replace the old text match with the new text while retaining the surrounding characters.

Requirements for PR merge:

  1. Unit & integration tests updated
  2. Necessary PR reviewers:
    • Backend
  3. Jira Ticket DEV-11608:
    • Link to this Pull-Request

Area for explaining above N/A when needed:

2. [ ] API documentation updated
4. [ ] Matview impact assessment completed
5. [ ] Frontend impact assessment completed
6. [ ] Data validation completed
7. [ ] Appropriate Operations ticket(s) created

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