removed jackson dependency, fixed invalid CQL order by syntax#31
removed jackson dependency, fixed invalid CQL order by syntax#31briandilley wants to merge 2 commits into
Conversation
|
It's a bad idea to use org.json. |
|
well, i could change it to use the latest version of Jackson. Jackson just seemed like overkill to me, but mainly it was using an older version. |
|
I've fixed the query builder syntax, thanks for that. As for the jackson 1.4 dependency, it is already pulled by cassandra-all anyway, so there's little point in changing it. We will try to minimize further the driver dependency in the future, but this will have to start by removing the dependency on cassandra-all which itself will require CASSANDRA-4623 first. As for Jackson, it's not used anywhere performance sensitive so I don't think it's worth bothering which version is used exactly. As a side note, I'd suggest avoiding unrelated changes in the same pull request. |
|
thanks. i did a single pull request because github wouldn't let me create more than one :/ |
Changed from an old version of jackson to json.org
Fixed order by statements for CQL select.