pybind/rados: allow bytearray as data input

[sileht]

rados python binding currently allow only immutable byte array (bytes).

This change adds support for mutable byte array (bytearray).

Signed-off-by: Mehdi Abaakouk sileht@sileht.net

[tchaikov]

see also #15136

[tchaikov]

This is a very good enhancement when ceph will raise the cython requirement. This avoids ton memory copy.

@sileht i'd suggest track this feature using tracker, and reference it from there instead of keeping this open until we drop the cython 0.19 support.

[sileht]

@tchaikov thank for the suggestion, I have created http://tracker.ceph.com/issues/21902

[stale]

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If you are a maintainer or core committer, please follow-up on this issue to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

[liewegas]

@tchaikov does this change still make sense?

[tchaikov]

@liewegas it makes sense. but in short term, i don't think we can ditch the support of RHEL/CentOS 7, and hence should continue building with cython 0.19, i just checked SCLo, it offers cython 0.25, see http://mirror.centos.org/centos/7/sclo/x86_64/sclo/sclo-python27/ , but there is no aarch64 builds there.

@sileht i am closing this PR at this moment. we will need to reopen it once we are able to use cython 0.20 or up. thanks!