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

Druid dashboard import/export. #1811

Merged
merged 6 commits into from
Jan 7, 2017
Merged

Druid dashboard import/export. #1811

merged 6 commits into from
Jan 7, 2017

Conversation

bkyryliuk
Copy link
Member

@bkyryliuk bkyryliuk commented Dec 10, 2016

Reviewers:

Testing:

  • unit tests are done
  • testing done on staging

@bkyryliuk bkyryliuk changed the title WIP. Druid dashboard export. Druid dashboard export. Dec 10, 2016
@bkyryliuk bkyryliuk changed the title Druid dashboard export. Druid dashboard import/export. Dec 10, 2016
@bkyryliuk
Copy link
Member Author

@mistercrunch - please take a look

@mistercrunch
Copy link
Member

Lgtm

@bkyryliuk
Copy link
Member Author

thanks !

@bkyryliuk bkyryliuk merged commit af872fa into apache:master Jan 7, 2017
@bkyryliuk bkyryliuk deleted the druid_dashboard_export branch January 7, 2017 00:48
@mistercrunch
Copy link
Member

This broke the build, reverting. We can bring it back when fixed

@bkyryliuk
Copy link
Member Author

@mistercrunch - my fault, will check this out.

SalehHindi pushed a commit to SalehHindi/superset that referenced this pull request Jun 9, 2017
* Druid dashboard import

* Fix tests.

* Parse params with trailing commas

* Resolved issue with datasource is not in DB yet when slice perms are set

* Finish rebase
@mistercrunch mistercrunch added the 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels label Feb 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.15.2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants