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

SUPP: Rename mapd to omniscidb backend #1866

Merged
merged 2 commits into from
Aug 15, 2019

Conversation

xmnlab
Copy link
Contributor

@xmnlab xmnlab commented Jun 29, 2019

depends on #1860

In this PR

  • Rename mapd backend to omniscidb

@xmnlab
Copy link
Contributor Author

xmnlab commented Aug 9, 2019

cc @niviksha

@xmnlab xmnlab marked this pull request as ready for review August 9, 2019 23:51
@xmnlab xmnlab requested a review from cpcloud August 9, 2019 23:51
@xmnlab
Copy link
Contributor Author

xmnlab commented Aug 12, 2019

@cpcloud it is done for a review :) thanks!

@xmnlab
Copy link
Contributor Author

xmnlab commented Aug 14, 2019

conflicts fixed. it is ready again for review.

@cpcloud
Copy link
Member

cpcloud commented Aug 15, 2019

@xmnlab I think that because this is a wholesale rename we should bump ibis to 2.0.0, thoughts?

@cpcloud cpcloud added this to the Next Major Release milestone Aug 15, 2019
Copy link
Member

@cpcloud cpcloud left a comment

Choose a reason for hiding this comment

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

LGTM!

@xmnlab
Copy link
Contributor Author

xmnlab commented Aug 15, 2019

@cpcloud thanks for the review! yes, you're right 2.0.0 sounds great!

@cpcloud cpcloud added feature Features or general enhancements omnisci labels Aug 15, 2019
@cpcloud cpcloud merged commit 9df2795 into ibis-project:master Aug 15, 2019
@xmnlab xmnlab deleted the rename-mapd-to-omniscidb branch August 15, 2019 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Features or general enhancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants