You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@zeppelin.apache.org by GitBox <gi...@apache.org> on 2020/03/26 12:02:07 UTC
[GitHub] [zeppelin] alexott opened a new pull request #3699: [ZEPPELIN-4378]
porting Cassandra interpreter to new driver
alexott opened a new pull request #3699: [ZEPPELIN-4378] porting Cassandra interpreter to new driver
URL: https://github.com/apache/zeppelin/pull/3699
### What is this PR for?
This PR refactors Cassandra interpreter to use new DataStax Java driver (4.x) that has new features & new architecture, that caused significant changes in the code base. But as result, we're getting:
* We get supported version of the driver (3.x line will only receive critical bug fixes)
* We get support for additional functionality of the DSE (DataStax Enterprise) - security, additional data types, etc.
* Improvements in the load balancing, etc.
### What type of PR is it?
Refactoring
### What is the Jira issue?
* ZEPPELIN-4378
### How should this be tested?
* Travis CI build: https://travis-ci.org/github/alexott/zeppelin/builds/667208139. The failing tests are the same as in the existing master
* Was tested manually in addition to the unit tests
### Questions:
* Does the licenses files need update? Done, minor updates
* Is there breaking changes for older versions? There are renames & deletion of the configuration properties, but this doesn't break existing setups
* Does this needs documentation? Documentation was updated
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] alexott commented on issue #3699: [ZEPPELIN-4378]
porting Cassandra interpreter to new driver
Posted by GitBox <gi...@apache.org>.
alexott commented on issue #3699: [ZEPPELIN-4378] porting Cassandra interpreter to new driver
URL: https://github.com/apache/zeppelin/pull/3699#issuecomment-604700276
Yep, the new driver requires big refactoring...
I didn’t change much from functional point of view, except removing retry
policy...
On Thu 26. Mar 2020 at 22:34, DuyHai DOAN <no...@github.com> wrote:
> This is a massive update. I need to check it out and review it locally !
>
> —
> You are receiving this because you authored the thread.
> Reply to this email directly, view it on GitHub
> <https://github.com/apache/zeppelin/pull/3699#issuecomment-604698981>, or
> unsubscribe
> <https://github.com/notifications/unsubscribe-auth/AAAHNBUE5GT54YBHLE54UATRJPC7VANCNFSM4LUFSVTQ>
> .
>
--
With best wishes, Alex Ott
http://alexott.net/
Twitter: alexott_en (English), alexott (Russian)
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] alexott commented on issue #3699: [ZEPPELIN-4378]
porting Cassandra interpreter to new driver
Posted by GitBox <gi...@apache.org>.
alexott commented on issue #3699: [ZEPPELIN-4378] porting Cassandra interpreter to new driver
URL: https://github.com/apache/zeppelin/pull/3699#issuecomment-604393047
@doanduyhai can you help review it?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] doanduyhai commented on issue #3699: [ZEPPELIN-4378]
porting Cassandra interpreter to new driver
Posted by GitBox <gi...@apache.org>.
doanduyhai commented on issue #3699: [ZEPPELIN-4378] porting Cassandra interpreter to new driver
URL: https://github.com/apache/zeppelin/pull/3699#issuecomment-604698981
This is a massive update. I need to check it out and review it locally !
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services
[GitHub] [zeppelin] alexott edited a comment on issue #3699: [ZEPPELIN-4378]
porting Cassandra interpreter to new driver
Posted by GitBox <gi...@apache.org>.
alexott edited a comment on issue #3699: [ZEPPELIN-4378] porting Cassandra interpreter to new driver
URL: https://github.com/apache/zeppelin/pull/3699#issuecomment-604700276
Yep, the new driver requires big refactoring...
I didn’t change much from functional point of view, except removing retry
policy...
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services