Skip to content

replace table_schema by schema_name#199

Merged
Brooke-white merged 1 commit into
aws:masterfrom
rmaxu:patch-1
Nov 27, 2023
Merged

replace table_schema by schema_name#199
Brooke-white merged 1 commit into
aws:masterfrom
rmaxu:patch-1

Conversation

@rmaxu
Copy link
Copy Markdown
Contributor

@rmaxu rmaxu commented Nov 21, 2023

Description

SVV_ALL_TABLES does not have table_schema column (https://docs.aws.amazon.com/redshift/latest/dg/r_SVV_ALL_TABLES.html)

Motivation and Context

Testing

Screenshots (if appropriate)

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Checklist

  • Local run of ./build.sh succeeds
  • Code changes have been run against the repository's pre-commit hooks
  • Commit messages follow Conventional Commit Specification
  • I have read the README document
  • I have added tests to cover my changes
  • I have run all unit tests using pytest test/unit and they are passing.
  • By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

SVV_ALL_TABLES does not have table_schema column
@Brooke-white Brooke-white self-requested a review November 27, 2023 18:12
Copy link
Copy Markdown
Contributor

@Brooke-white Brooke-white left a comment

Choose a reason for hiding this comment

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

Hi @rmaxu , thank you for identifying and creating a fix for #198. We appreciate your contribution 🚀

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