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

NFThinCassandraDaemon should work with JSVC like Cassandra does #20

Merged
merged 2 commits into from May 11, 2012
Merged

NFThinCassandraDaemon should work with JSVC like Cassandra does #20

merged 2 commits into from May 11, 2012

Conversation

codyaray
Copy link
Contributor

@codyaray codyaray commented May 8, 2012

Out of the (debian) box, Cassandra runs with JSVC. We want to run a lightly modified version that supports Priam, using the NFThinCassandraDaemon. However, the current implementation doesn't work since JSVC hooks into init(String[]) rather than main(String[]). This patch allows running either way.

@Vijay2win
Copy link
Contributor

There is some interesting discussion about JSVC in https://issues.apache.org/jira/browse/CASSANDRA-3991 at this point i am not sure if we need it.

@sorenvind
Copy link

I can confirm that this fix makes Priam behave nicely with Cassandra on Debian Squeeze.

Vijay2win added a commit that referenced this pull request May 11, 2012
NFThinCassandraDaemon should work with JSVC like Cassandra does
@Vijay2win Vijay2win merged commit 79e974a into Netflix:master May 11, 2012
@Vijay2win
Copy link
Contributor

Thanks for the patch and the test.

schakrovorthy added a commit that referenced this pull request May 6, 2020
…rthy-3.x

CASS-1731 Fixing code formatting errors.
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.

3 participants