]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commit
rgw_file: unify allocations of request id 28108/head
authorMatt Benjamin <mbenjamin@redhat.com>
Wed, 15 May 2019 13:31:03 +0000 (09:31 -0400)
committerMatt Benjamin <mbenjamin@redhat.com>
Wed, 15 May 2019 14:07:59 +0000 (10:07 -0400)
commit51356347a3400bbf7ea846c3e1196aeb7be0ab60
tree1d2d3bac39baaba6daa56c3cca6908804c1e3c2c
parente2cdb04ba6202a7e45299427f3cbc654853bbd7f
rgw_file: unify allocations of request id

The fix by Tao Chen <chentao@umcloud.com> for construction of the
request id in RGWWriteRequests/RGWContinuedReq paths--just writes,
effectively--revealed that RGWRequest::id was being set to zero
in general, while a valid request id was being assigned to
req_state.

This implies that code which the RGWRequest::id of non-write requests
would still use an invalid value, so avoid that by initializing
RGWRequest::id in all RGWLibRequests, and propagating its value
to req_state.  This still works as intended for RGWWriteRequest.

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
src/rgw/librgw.cc
src/rgw/rgw_file.h
src/rgw/rgw_lib.h