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
python-rados: extends ReadOp/WriteOp API #9944
Conversation
* rados_aio_write_op_operate * rados_write_op_omap_set * rados_read_op_set_flags * rados_aio_read_op_operate Upstream: ceph/ceph#9944
* rados_aio_write_op_operate * rados_write_op_omap_set * rados_read_op_set_flags * rados_aio_read_op_operate Upstream: ceph/ceph#9944
* rados_aio_write_op_operate * rados_write_op_omap_set * rados_read_op_set_flags * rados_aio_read_op_operate Upstream: ceph/ceph#9944
acfbe38
to
35e5606
Compare
Any new about merging this ? |
sorry, I meant to look at this a lot sooner. Will review today |
jenkins: retest this please |
@requires(('to_write', bytes)) | ||
def append(self, to_write): | ||
""" | ||
Write whole object, atomically replacing it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
accidentally used write_full description
looks good, just a couple minor comments. could you add a signed-off-by line to your commit messages? |
This change adds python API for the following C methods: * rados_aio_write_op_operate * rados_write_op_omap_set * rados_read_op_set_flags * rados_aio_read_op_operate Signed-off-by: Mehdi Abaakouk <sileht@redhat.com>
* rados_write_op_create * rados_write_op_append * rados_write_op_write_full * rados_write_op_write * rados_write_op_remove * rados_write_op_truncate * rados_write_op_zero Signed-off-by: Mehdi Abaakouk <sileht@redhat.com>
I have fixed the comments |
thanks! |
This change extends python-rados ReadOp/WriteOp API for the following C methods: