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

Add kylin datasource. #605

Closed
wants to merge 2 commits into from
Closed

Conversation

rocky1001
Copy link
Contributor

Add kylin datasource support (combined with this repo as kylin orm: https://github.com/rocky1001/pykylin/tree/caravel-kylin)

Because 'timestamp' is the sql keywords that kylin not supported, use '_timestamp' as the label when using kylin datasource.

@@ -1,112 +1,12 @@
Caravel
=========
<img src="http://i.imgur.com/H0Kyvyi.jpg" style="border-radius: 20px; box-shadow:5px 5px 5px gray;" alt="Caravel" width="500"/>

Copy link
Member

Choose a reason for hiding this comment

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

Nope.

@mistercrunch
Copy link
Member

Can you rebase? There are new hooks to customize for specific backends
https://github.com/airbnb/caravel/blob/master/caravel/models.py#L440

@mistercrunch mistercrunch added the inactive Inactive for >= 30 days label Jun 13, 2016
@mistercrunch
Copy link
Member

Please re-open a PR that takes into account recent code changes.

@rocky1001
Copy link
Contributor Author

​​Ok, I will try to add Apache Kylin backends with the latest code and
re-open a PR,

Thank you.

2016-06-14 0:38 GMT+08:00 Maxime Beauchemin notifications@github.com:

Please re-open a PR that takes into account recent code changes.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#605 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ADIR7NVNK3O0le8oeBagp_Gn6auErrltks5qLYeVgaJpZM4Iz7r2
.

@nezhazheng
Copy link

Any progress? @rocky1001

1 similar comment
@tianlinzx
Copy link

Any progress? @rocky1001

zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 17, 2021
Bumps [@vx/pattern](https://github.com/hshoff/vx) from 0.0.196 to 0.0.197.
- [Release notes](https://github.com/hshoff/vx/releases)
- [Changelog](https://github.com/hshoff/vx/blob/master/CHANGELOG.md)
- [Commits](airbnb/visx@v0.0.196...v0.0.197)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

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 pull request Nov 24, 2021
Bumps [@vx/pattern](https://github.com/hshoff/vx) from 0.0.196 to 0.0.197.
- [Release notes](https://github.com/hshoff/vx/releases)
- [Changelog](https://github.com/hshoff/vx/blob/master/CHANGELOG.md)
- [Commits](airbnb/visx@v0.0.196...v0.0.197)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

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 pull request Nov 25, 2021
Bumps [@vx/pattern](https://github.com/hshoff/vx) from 0.0.196 to 0.0.197.
- [Release notes](https://github.com/hshoff/vx/releases)
- [Changelog](https://github.com/hshoff/vx/blob/master/CHANGELOG.md)
- [Commits](airbnb/visx@v0.0.196...v0.0.197)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

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 pull request Nov 26, 2021
Bumps [@vx/pattern](https://github.com/hshoff/vx) from 0.0.196 to 0.0.197.
- [Release notes](https://github.com/hshoff/vx/releases)
- [Changelog](https://github.com/hshoff/vx/blob/master/CHANGELOG.md)
- [Commits](airbnb/visx@v0.0.196...v0.0.197)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inactive Inactive for >= 30 days
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants