cassandra-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Aleksey Yeschenko (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (CASSANDRA-10653) Remove dependency on jgrapht for UDT resolution
Date Tue, 17 Nov 2015 13:56:11 GMT

    [ https://issues.apache.org/jira/browse/CASSANDRA-10653?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15008669#comment-15008669
] 

Aleksey Yeschenko commented on CASSANDRA-10653:
-----------------------------------------------

Pushed a branch to my repo, tests running:

||branch||testall||dtest||
|[10653|https://github.com/iamaleksey/cassandra/tree/10653]|[testall|http://cassci.datastax.com/view/Dev/view/iamaleksey/job/iamaleksey-10653-testall]|[dtest|http://cassci.datastax.com/view/Dev/view/iamaleksey/job/iamaleksey-10653-dtest]|

It's not even taking that much more code, when hand-rolled. Now that the driver doesn't use
jgrapht, it's an utter overkill to keep it (though I still maintain that v1 was ugly as hell).

Test coverage piggybacks on existing {{UserTypesTest}}, {{LegacyMigratorTest}} and dtests.
CASSANDRA-10364 will be adding more, including more complex UDT graphs.

> Remove dependency on jgrapht for UDT resolution
> -----------------------------------------------
>
>                 Key: CASSANDRA-10653
>                 URL: https://issues.apache.org/jira/browse/CASSANDRA-10653
>             Project: Cassandra
>          Issue Type: Improvement
>          Components: Distributed Metadata
>            Reporter: Aleksey Yeschenko
>            Assignee: Aleksey Yeschenko
>            Priority: Minor
>             Fix For: 3.0.x
>
>
> Now that the java-driver no longer pulls it as a dependency, it is silly to pull a whole
library for resolving UDTs dependencies.
> Should rewrite the resolution code without jgrapht (maybe reuse whatever code java-driver
ended up writing).



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message