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

Downgrade to avatica-server 1.8.0, skip avatica-core. #3813

Merged
merged 1 commit into from
Jan 4, 2017

Conversation

gianm
Copy link
Contributor

@gianm gianm commented Jan 3, 2017

This matches the version bundled by Calcite 1.10.0.

I think this should fix https://groups.google.com/d/topic/druid-user/_fzmIhK14Kc/discussion, which is caused by loading some avatica 1.9.0 jars on the classpath.

@gianm gianm added the Bug label Jan 3, 2017
@gianm gianm added this to the 0.10.0 milestone Jan 3, 2017
This matches the version bundled by Calcite 1.10.0.
@fjy
Copy link
Contributor

fjy commented Jan 3, 2017

👍

@fjy fjy merged commit 1f35120 into apache:master Jan 4, 2017
@fjy
Copy link
Contributor

fjy commented Jan 4, 2017

I just realized I merged this without a 2nd +1 because in my mind we were doing single +1s for bugs but that might have for docs, so someone else should review :P

@gianm
Copy link
Contributor Author

gianm commented Jan 4, 2017

I don't think we have an exception for bugs.

@jon-wei
Copy link
Contributor

jon-wei commented Jan 4, 2017

👍

@gianm gianm deleted the fix-jackson branch January 4, 2017 04:26
@mario4tier
Copy link

This fix my problem reported in https://groups.google.com/d/topic/druid-user/_fzmIhK14Kc/discussion

Thanks.

@gianm
Copy link
Contributor Author

gianm commented Jan 4, 2017

Thanks for confirming @Fortier!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants