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

Update Cassandra 0.7 to latest API. #24

Closed
wants to merge 1 commit into from

Conversation

strickli
Copy link

Updated Cassandra0.7 plugin to use API for 0.7 release: replaced byte[]
style args with ByteBuffer. Used cassandra.util.ByterBufferUtil to
munge between the two formats.

style args with ByteBuffer.  Used cassandra.util.ByterBufferUtil to
munge between the two formats.
@heapxor
Copy link

heapxor commented Feb 9, 2011

thanks.

@brianfrankcooper
Copy link
Owner

Hi,

Sorry for the delay in getting to this. I just merged another pull request that makes similar changes. Is your patch still necessary? Thanks for submitting it in any event!

Brian

@strickli
Copy link
Author

Hi,

Sorry for the delay in getting to this. I just merged another pull request
that makes similar changes. Is your patch still necessary? Thanks for
submitting it in any event!

Brian

Brian:
There was a patch that predated mine by three days that appeared to make
the same fixes. If I recall correctly, they used a self-written
translation routine for buffers, whereas I pulled the same functionality
out of Cassie's library.

In any case, I imagine my patch is redundant. Thanks for your good work!

-mcs

Michael Stricklin | strickli@arlut.utexas.edu | 512-835-3071
Applied Research Laboratories, UT Austin

@msf
Copy link

msf commented Jun 22, 2011

this is still required to make YCSB build the dbcompile-cassandra-0.7 target.
Please pull this.

@brianfrankcooper
Copy link
Owner

@msf - Are you sure I need to pull this? I just built ant dbcompile-cassandra-0.7 with a newly downloaded apache-cassandra-0.7.6-2 without any problems.

@ghost
Copy link

ghost commented Aug 24, 2011

Hello,

dbcompile-cassandra-0.7 is working with 0.7.8. It might have gotten fixed as a part of gh-34. I'm closing this ticket.

Thanks!
--Michi

@ghost ghost closed this Aug 24, 2011
hse-root referenced this pull request in hse-project/hse-ycsb Aug 21, 2020
…re-dependency to v0.17.0.2.1-hse-1.8.0

* commit 'e859781a24d2726f3e006105c44721de14605159':
  Add default-jre as an install dependency in debian package
This pull request was closed.
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