Incubator-superset: Datasource edit doesn't save schema, only saves new dataset name

Created on 22 Oct 2020  路  7Comments  路  Source: apache/incubator-superset

On chart explore, when you click on change dataset, and try to edit the datasource (table) name, the table name change works as expected but unless you edit schema, the existing schema doesn't get saved causing the chart to break.

Expected results

Both schema and table name should change.

Actual results

Only table name changes.

Screenshots

Step 1: my table name (before change)
Screen Shot 2020-10-21 at 6 38 34 PM

Step 2: try to edit table name, schema is same - do not edit schema
Screen Shot 2020-10-21 at 6 37 46 PM

Step 3: This is the result: New table name is displayed with NO SCHEMA. if you run chart, it breaks.
Screen Shot 2020-10-21 at 6 37 53 PM

How to reproduce the bug

  1. Go to 'Chart Explore'
  2. Click on 'Change Dataset'
  3. Enter new table name, keep same schema
  4. Click on Save
  5. See issue

Environment

(please complete the following information):

  • superset version: master

    Checklist

Make sure these boxes are checked before submitting your issue - thank you!

  • [x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • [x] I have reproduced the issue with at least the latest released version of superset.
  • [x] I have checked the issue tracker for the same issue and I haven't found one similar.
#bug .explore blocking!

Most helpful comment

All 7 comments

Issue-Label Bot is automatically applying the label #bug to this issue, with a confidence of 0.95. Please mark this comment with :thumbsup: or :thumbsdown: to give our bot feedback!

Links: app homepage, dashboard and code for this bot.

@lilykuang

Closing as the fix is merged.

This issue has cropped up again on our side

@benceorlai the step I tried to reproduce this issue
https://www.loom.com/share/494cf29e697841f5b6c038c2642892bc
It doesn't seem to have issue. did i miss anything?

Was this page helpful?
0 / 5 - 0 ratings