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

client/Client.cc: fix/silence "logically dead code" CID-Error #291

Merged
merged 1 commit into from May 16, 2013

Conversation

dalgaaf
Copy link
Contributor

@dalgaaf dalgaaf commented May 16, 2013

Fix handling of 'safe' and the conditions after calling file_flush().

CID 1019548 (#1 of 1): Logically dead code (DEADCODE)
dead_error_begin: Execution cannot reach this statement "flock.Unlock();"

Signed-off-by: Danny Al-Gaaf danny.al-gaaf@bisect.de

v2 of pull request #288 (use bool instead of int)

Fix handling of 'safe' and the conditions after calling file_flush().

CID 1019548 (#1 of 1): Logically dead code (DEADCODE)
  dead_error_begin: Execution cannot reach this statement "flock.Unlock();"

Signed-off-by: Danny Al-Gaaf <danny.al-gaaf@bisect.de>
liewegas pushed a commit that referenced this pull request May 16, 2013
client/Client.cc: fix/silence "logically dead code" CID-Error

Even money that this satisfies the coverity gods...

Reviewed-by: Sage Weil <sage@inktank.com>
@liewegas liewegas merged commit 2df6e37 into ceph:master May 16, 2013
@dalgaaf dalgaaf deleted the wip-da-CID-1019548 branch May 17, 2013 11:46
liewegas pushed a commit that referenced this pull request Dec 14, 2016
rgw: set data extra pool correctly

Backport: next, firefly
Reviewed-by: Sage Weil <sage@redhat.com>
liewegas pushed a commit that referenced this pull request Dec 14, 2016
add erasure code repair / scrub tests (take 2)

Reviewed-by: David Zafman <dzafman@redhat.com>
jan--f pushed a commit to jan--f/ceph that referenced this pull request May 28, 2019
qa/tasks: iscsiadm should listen on non lo ip address
liewegas added a commit to liewegas/ceph that referenced this pull request Nov 5, 2021
community/team: add a few missing people; alphabetize
andriytk pushed a commit to andriytk/cortx-rgw that referenced this pull request Jun 15, 2022
…on (ceph#291) (ceph#292)

* rgw_sal_motr: [CORTX-31877] Support Stat counting for Multipart & Obj OverWrite case (ceph#286)

* rgw_sal_motr: [CORTX-31877] Support Stats for Multipart & Obj OverWrite case

In multipart upload, each part upload increments the total object size by part size
and count by one. When the complete multipart is called, the object count is reset to
consider all parts as single object.
On abort size of each part is subtracted and the object count is decremented by number
of parts as well.

Object overwrite case is also hanlded with the same PR which decrement the size of old
obj and increment the size of new obj in case of unversioned bucket. Obj count will
remain the same.

* rgw_sal_motr: [CORTX-31877] handle stats on delete marker delete

Delete marker creating does not increment the stats but deletion of
delete marker was causing object count to go down by 1.
Addressed the same by addition of a delete marker check.

Signed-off-by: Sumedh Anantrao Kulkarni <sumedh.a.kulkarni@seagate.com>

* rgw, rgw_sal_motr: [CORTX-31836] calculate actual size for all types of objects (ceph#287)

Size Rounded is the actual size occupied by an object on disk including
padding required as per the unit and block size.
size_rounded parameter is added in bucket_dir_entry as well as in sal motr
object and part related classes to store the calculated size.

Signed-off-by: Sachin Punadikar <sachin.punadikar@seagate.com>
Signed-off-by: Diwakar92 <diwakar.kumar@seagate.com>
Signed-off-by: Sumedh Anantrao Kulkarni <sumedh.a.kulkarni@seagate.com>
tobias-urdin pushed a commit to tobias-urdin/ceph that referenced this pull request Aug 2, 2023
Add a test case for continuation token in list objects v2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants