Skip to content

Conversation

@KSDaemon
Copy link
Member

@KSDaemon KSDaemon commented Mar 19, 2025

This PR supersedes the original PR by @emmanuel with conflict resolving and some polishment

Check List

  • Tests have been run in packages where changes made if available
  • Linter has been run for changed code
  • Tests for the changes have been added if not covered yet
  • Docs have been added / updated if required

Fixes #3876.

emmanuel and others added 3 commits January 4, 2022 23:17
…cube.js into emmanuel-fix/3876-redshift-schema-query

# Conflicts:
#	packages/cubejs-redshift-driver/src/RedshiftDriver.ts
@KSDaemon KSDaemon marked this pull request as ready for review March 19, 2025 17:29
@KSDaemon KSDaemon requested a review from a team as a code owner March 19, 2025 17:29
@emmanuel
Copy link
Contributor

Awesome, thanks for getting this across the line!

@KSDaemon KSDaemon merged commit beaf8eb into master Mar 21, 2025
64 checks passed
@KSDaemon KSDaemon deleted the emmanuel-fix/3876-redshift-schema-query branch March 21, 2025 13:58
marianore-muttdata pushed a commit to MuttData/cube that referenced this pull request Jun 17, 2025
* fix(redshift-driver): Use Redshift schema query.

Fixes cube-js#3876.

* fix(redshift-driver): Use Redshift-specific schema query

---------

Co-authored-by: Emmanuel Gomez <[email protected]>
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.

Use Redshift-appropriate query for information schema discovery

5 participants