Skip to content

Conversation

@lnorregaard
Copy link

This is the code needed to run under java 8. Also Cassandra is now using the CQL communication instead of Thrift.
Spring version have been upgraded to version 4.
The Spring 4 security also introduced that the j_username and j_password have changed to username and password

A MockMVCTest is failing in the web sub project. This is outcommented at the moment as it seems to be a problem with mockmvc. I will look into it later

whelan and others added 30 commits November 20, 2015 10:48
@lnorregaard lnorregaard changed the title Update to Spring 4 Update to datastax java driver and Spring 4 and also Java 8 code Dec 1, 2015
public LoadBalancingPolicy parseLbPolicy(String loadBalancingPolicyString) throws InstantiationException,
IllegalAccessException, ClassNotFoundException, NoSuchMethodException, SecurityException,
IllegalArgumentException, InvocationTargetException {
String lb_regex = "([a-zA-Z]*Policy)(\\()(.*)(\\))";

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is this regex match all the parsers which you are trying to parse.
I guess the Policy part which you are using might not give the exact part.
if not getting my point please let me know

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not sure what you mean?
It tries to get the load balancing policy out together with the parameters for the load balancing

@DRogozenski
Copy link

You had the right idea on updating these dependencies, it's a shame they were never merged into the master branch.

I'd like to see this project be updated to the point where it could work today. I've tried updating the package dependencies and maven plugin versions, but it seems there's more to it than that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants