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.

Sign up for GitHub

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

Jump to bottom

Add Python Interface #66

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

Add Python Interface #66

wants to merge 4 commits into from

Conversation

JCoym
Copy link
Contributor

@JCoym JCoym commented Oct 14, 2019 •
edited

This pull request adds a Python interface for the KV, Object, Distributed Object and Item clients.

@michaelkuhn michaelkuhn self-assigned this Mar 20, 2020
@michaelkuhn
Copy link
Member

Could you please rebase and squash this? This is currently split up into a few too many commits.

Have you had a chance to track down the bug you mentioned or is it still present in this version?

@JCoym JCoym force-pushed the python branch 2 times, most recently from 2cb0a3c to db56bb8 Compare March 23, 2020 01:06
@JCoym
Copy link
Contributor Author

JCoym commented Mar 23, 2020

The error still remains unresolved.
When writing >16 Bytes using the Python Object Client, the data correctly arrives in j_object_write of the C interface(Checked with a print in j_object_create), but then in the saved file, and therefore also in following reads, the first 8 Bytes are corrupted.

@JCoym
Copy link
Contributor Author

JCoym commented Dec 20, 2020 •
edited

The error is now resolved with the newest commit 44e13b4

@michaelkuhn michaelkuhn mentioned this pull request Apr 27, 2021
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@michaelkuhn michaelkuhn

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@JCoym @michaelkuhn