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

Fix incorrect ReleasePrimitiveArrayCritical jni call #46

Merged
merged 1 commit into from Aug 19, 2014
Merged

Fix incorrect ReleasePrimitiveArrayCritical jni call #46

merged 1 commit into from Aug 19, 2014

Conversation

sunxiaoguang
Copy link
Contributor

Fix incorrect ReleasePrimitiveArrayCritical jni call

@pluradj pluradj mentioned this pull request Aug 18, 2014
@pluradj
Copy link

pluradj commented Aug 18, 2014

This fixes a problem where Apache Cassandra 2.x crashes at start up when running with IBM Java 7.
https://issues.apache.org/jira/browse/CASSANDRA-7573

@sunxiaoguang
Copy link
Contributor Author

Cool, I'm glad this fix can actually help people.

On 2014Äê8ÔÂ19ÈÕ, at ÉÏÎç2:52, pluradj notifications@github.com wrote:

This fixes a problem where Apache Cassandra 2.x crashes at start up when running with IBM Java 7.
https://issues.apache.org/jira/browse/CASSANDRA-7573

¡ª
Reply to this email directly or view it on GitHub.

jpountz added a commit that referenced this pull request Aug 19, 2014
Fix incorrect ReleasePrimitiveArrayCritical jni call
@jpountz jpountz merged commit 8d1170d into lz4:master Aug 19, 2014
@jpountz
Copy link
Collaborator

jpountz commented Aug 19, 2014

Good catch! Thanks for the fix.

@pluradj
Copy link

pluradj commented Aug 19, 2014

@sunxiaoguang Yes, definitely, thanks for the fix.

@jpountz Thanks for getting this merged. Do you have a time line in place for a new release? I want to make sure that Cassandra picks up this fix.

@gordysc
Copy link

gordysc commented Oct 9, 2014

@jpountz, I see the fix for this is delivered, but not captured in an official version in Maven Central (the library has to be built from source to include the fix). As @pluradj asked, do we have a timeline of when this might happen? Thanks!

@jpountz
Copy link
Collaborator

jpountz commented Oct 11, 2014

Sorry for the delay, I will try to do a release next week!

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.

None yet

4 participants