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

Issue in adding tables for Redshift #336

Closed
shahneil88 opened this issue Apr 13, 2016 · 4 comments · Fixed by #330 · May be fixed by EC-Snyk/incubator-superset#81 or ajesse11x/incubator-superset#119
Closed

Issue in adding tables for Redshift #336

shahneil88 opened this issue Apr 13, 2016 · 4 comments · Fixed by #330 · May be fixed by EC-Snyk/incubator-superset#81 or ajesse11x/incubator-superset#119

Comments

@shahneil88
Copy link

Hi,

  1. I created a connection using
    redshift+psycopg2://username:password@host:5439/dev
  2. I clicked on Test Connection. It says "Connection seems ok!"
  3. I save it and go back.
  4. I click on Sources -> Tables -> New table
  5. Give name of table and Database as my newly created Database and click on save. It gives me below error.

screen shot 2016-04-13 at 12 18 28 pm

  1. I go back to my Database and see that connection string has become

redshift+psycopg2://username:XXXXXXXXXX@host:5439/dev
I believe password has been masked.
But now when i click on Test Connection it gives error

screen shot 2016-04-13 at 12 20 59 pm

  1. When I click on table name, which I created in step 5, it redirects me to error page

screen shot 2016-04-13 at 12 32 38 pm

Not sure, what am I doing wrong.

@kingo55
Copy link
Contributor

kingo55 commented Apr 13, 2016

+1

@shahneil88 - I believe it masks the password with Xs and uses the mask as the password when you test the connection. Just replace the Xs with your actual password and try "Test Connection" again.

As for adding tables, can you add tables manually after updating your search path? E.g.

Sources > Tables > Add new Sqla table

Type the table name in eg. if it's "atomic.impressions" with the schema, just use "impressions".

In general though, this is a bug I've also noticed.

@shahneil88
Copy link
Author

@kingo55 -

I did try adding tables. I cannot add tables even from public schema.

@kingo55
Copy link
Contributor

kingo55 commented Apr 13, 2016

Interesting... Does the user have access to the table? Do they also have usage on the schema?

I had a similar issue where I managed to fix it by granting usage of the schema to the user.

@shahneil88
Copy link
Author

Yes, My Redshift DB has only 1 user. Admin user.
I have provided those credentials in caravel connections DB.

@deniszgonjanin deniszgonjanin mentioned this issue Apr 13, 2016
graceguo-supercat pushed a commit to graceguo-supercat/superset that referenced this issue Oct 4, 2021
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 8.1.0 to 9.0.0.
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@8.1.0...9.0.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 8.1.0 to 9.0.0.
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@8.1.0...9.0.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 17, 2021
Bumps [set-value](https://github.com/jonschlinkert/set-value) from 0.4.3 to 2.0.1. **This update includes a security fix.**
- [Release notes](https://github.com/jonschlinkert/set-value/releases)
- [Commits](jonschlinkert/set-value@0.4.3...2.0.1)

Signed-off-by: dependabot-preview[bot] <[email protected]>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 8.1.0 to 9.0.0.
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@8.1.0...9.0.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 24, 2021
Bumps [set-value](https://github.com/jonschlinkert/set-value) from 0.4.3 to 2.0.1. **This update includes a security fix.**
- [Release notes](https://github.com/jonschlinkert/set-value/releases)
- [Commits](jonschlinkert/set-value@0.4.3...2.0.1)

Signed-off-by: dependabot-preview[bot] <[email protected]>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 8.1.0 to 9.0.0.
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@8.1.0...9.0.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 25, 2021
Bumps [set-value](https://github.com/jonschlinkert/set-value) from 0.4.3 to 2.0.1. **This update includes a security fix.**
- [Release notes](https://github.com/jonschlinkert/set-value/releases)
- [Commits](jonschlinkert/set-value@0.4.3...2.0.1)

Signed-off-by: dependabot-preview[bot] <[email protected]>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
Bumps [fs-extra](https://github.com/jprichardson/node-fs-extra) from 8.1.0 to 9.0.0.
- [Release notes](https://github.com/jprichardson/node-fs-extra/releases)
- [Changelog](https://github.com/jprichardson/node-fs-extra/blob/master/CHANGELOG.md)
- [Commits](jprichardson/node-fs-extra@8.1.0...9.0.0)

Signed-off-by: dependabot-preview[bot] <[email protected]>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this issue Nov 26, 2021
Bumps [set-value](https://github.com/jonschlinkert/set-value) from 0.4.3 to 2.0.1. **This update includes a security fix.**
- [Release notes](https://github.com/jonschlinkert/set-value/releases)
- [Commits](jonschlinkert/set-value@0.4.3...2.0.1)

Signed-off-by: dependabot-preview[bot] <[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
2 participants