Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

can not set cassandra port configurition #20

Open
jixuan1989 opened this issue Aug 17, 2012 · 3 comments
Open

can not set cassandra port configurition #20

jixuan1989 opened this issue Aug 17, 2012 · 3 comments
Assignees

Comments

@jixuan1989
Copy link

we can run virgil by bin/virgil -host and
the listening port of my cassandra node is 9170 instead of 9160...
but I can't set the port in virgil...so I can't run virgil to listen my cassandra cluster...

@ghost ghost assigned boneill42 Aug 17, 2012
@boneill42
Copy link
Contributor

I'll check to make sure the port is being passed through. We may have lost that parameter in the conversion to dropwizard.

-brian

On Aug 17, 2012, at 2:34 AM, sainthxd wrote:

we can run virgil by bin/virgil -host and
the listening port of my cassandra node is 9170 instead of 9160...
but I can't set the port in virgil...so I can't run virgil to listen my cassandra cluster...


Reply to this email directly or view it on GitHub.

Brian ONeill
Lead Architect, Health Market Science (http://healthmarketscience.com)
mobile:215.588.6024
blog: http://weblogs.java.net/blog/boneill42/
blog: http://brianoneill.blogspot.com/

@boneill42
Copy link
Contributor

Looks like it is now pulled from a system parameter in VirgilConfiguration.java:

public static int getPort() {
    return Integer.parseInt(System.getProperty(VirgilConfiguration.CASSANDRA_PORT_PROPERTY));
}

That is certainly less than ideal. I know we had issues moving it to a command line parameter though because of dropwizard. I'll see if dropwizard has fixed the issue. If not, we can move it into the yaml file.

@boneill42
Copy link
Contributor

BTW, the system parameter is:

public final static String CASSANDRA_PORT_PROPERTY = "virgil.cassandra_port";

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

No branches or pull requests

2 participants