Uh oh!
There was an error while loading. Please reload this page.
[ZEPPELIN-881] Apache Ignite version updated to 1.6 - #916
Conversation
jongyoul
commented
May 26, 2016
Simple! Great! And I think Ignite JDBC interpreter doesn't have any special thing. How do you think it merges into JDBC interpreter? |
jongyoul
commented
May 26, 2016
Without the issue above, this PR looks good to me. |
agura
commented
May 26, 2016
@jongyoul Ignite JDBC interpreter has some specific options, so I need to investigate possibility of usage generic JDBC interpreter. I think the best way is to merge this PR in order to include new drivers in 0.6.0 release. |
jongyoul
commented
May 26, 2016
@agura I'll thank you if you help merge Ignite JDBC into generic JDBC interpreter. And about this PR, it looks good to me. Merging if there's no more discussion. |
agura
commented
May 30, 2016
Guys, can anybody merge this PR into master branch? |
@agura Hi, one of CI test fails. Could you please check it? |
agura
commented
May 30, 2016
@jongyoul How to rerun tests? |
jongyoul
commented
May 30, 2016
@agura Close and re-open this PR |
jongyoul
commented
May 30, 2016
@Leemoonsoo Could you please check that failure due to flaky test? |
@jongyoul first thing to do is to check JIRA The CI failure is not related to the changes and I think is already tracked under https://issues.apache.org/jira/browse/ZEPPELIN-888 so I have added this failure there as well. Looks good to me, merging if there is no further discussion |
agura
commented
May 31, 2016
Thx, guys! |
jongyoul
commented
May 31, 2016
We also appreciate your contribution! |
### What is this PR for? While Apache Ignite version in Zeppelin was bumped up by #916, but ignite build flag in [README.md](https://github.com/apache/incubator-zeppelin/blob/master/README.md#ignite-interpreter) is outdated so far. So I just changed `1.1.0-incubating` -> `1.6.0`. ### What type of PR is it? Hot Fix ### Todos ### What is the Jira issue? ### How should this be tested? Just see [here](https://github.com/apache/incubator-zeppelin/blob/master/README.md#ignite-interpreter). ### Screenshots (if appropriate) ### Questions: * Does the licenses files need update? No * Is there breaking changes for older versions? No * Does this needs documentation? No Author: AhyoungRyu <fbdkdud93@hanmail.net> Closes#970 from AhyoungRyu/update/README.md and squashes the following commits: 20d0155 [AhyoungRyu] Update README.md for fixing ignite flag
What is this PR for?
Apache Ignite 1.6 contains JDBC driver improvements that affect Zeppelin Ignite interpreter user experience.
What type of PR is it?
[Improvement]
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-881