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

added support of missing functionalities of rocksdb #10

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ossdev07
Copy link

modified existing functionalities and added missing as per latest rocksdb(rocksdb_5.9.2)

modified existing functionalities and added missing as per latest rocksdb(rocksdb_5.9.2)
@ossdev07
Copy link
Author

@chirino as all the checks have been passed successfully. So could you please suggest us the next steps to be followed or done.

@ossdev07
Copy link
Author

can any maintainer have a look on these changes..

@ossdev07
Copy link
Author

ossdev07 commented Apr 2, 2019

Please merge this also as hawtjni changes are merged
fusesource/hawtjni#50

@ossdev07
Copy link
Author

ossdev07 commented Apr 2, 2019

@gnodet can you please have a look on these changes also
hawtjni changes are already merged
fusesource/hawtjni#50

@gnodet
Copy link
Member

gnodet commented Apr 2, 2019

I don't see any usage of shared_ptr in your commit ? How does this PR relates to the hawtjni PR ?

@ossdev07
Copy link
Author

ossdev07 commented Apr 8, 2019

Hi @gnodet

Yes, this PR does not relate with the hawtjni PR.

The PR add the new functionalites which is present in rocksdb but not currently supported by rocksdbjni, functionlities like Keymayexist, Single delete for write Batch

Please consider the PR to merge.

@ossdev07
Copy link
Author

@gnodet this is long pending PR to provide support of missing functionalities in rocksdbjni and present in rocksdb
it would be helpful for all rocksdbJni users, Please consider the PR to merge.

@ossdev07
Copy link
Author

ossdev07 commented May 2, 2019

Hi @gnodet

I am curious to know when are you planning in merging the PR. If any help or information required from our side, do let us know.

It will be helpful for the rockdbjni users, to see the support of the missing functionalities of the rocksdbjni

@ossdev07
Copy link
Author

@gnodet Any update on the PR merge status.

@ossdev07
Copy link
Author

@gnodet can you review this PR to be merged

@ossdev07
Copy link
Author

ossdev07 commented Oct 7, 2019

can someone have a look on this PR in which i tried to align with RocksDB

@gnodet
Copy link
Member

gnodet commented Nov 15, 2019

Maybe you should investigate switching to the official rocksdb java support which is available since RocksDb 3.5.0, 4 years ago:
http://repo1.maven.org/maven2/org/rocksdb/rocksdbjni/

@ossdev07
Copy link
Author

Maybe you should investigate switching to the official rocksdb java support which is available since RocksDb 3.5.0, 4 years ago:
http://repo1.maven.org/maven2/org/rocksdb/rocksdbjni/

@gnodet i am not able to access this link

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.

2 participants