Project

General

Profile

Bug #9342

Different implementation for PGTransaction::get_bytes_written between RPGTransaction and ECTransaction.

Added by jianpeng ma over 5 years ago. Updated almost 3 years ago.

Status:
Resolved
Priority:
Low
Assignee:
Category:
OSD
Target version:
-
% Done:

0%

Source:
other
Tags:
Backport:
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Pull request ID:
Crash signature:

Description

The l_osd_op_w_inb use this value. So the perf-dump display different value when write same object to erasure-pool/replicated-pool.
I think the implementation of ECTransaction is right. It should record the size of data which client wrote.

History

#1 Updated by Loic Dachary over 5 years ago

  • Priority changed from Normal to High

Ma Jianpeng, if you are working on it would you be so kind as to change from New to In Progress ?

#2 Updated by jianpeng ma over 5 years ago

Loic, i think i have no permit to change stats.

The pull request:
https://github.com/ceph/ceph/pull/2400

#3 Updated by Loic Dachary over 5 years ago

  • Status changed from New to In Progress

Here you go :-)

#4 Updated by Samuel Just over 5 years ago

Hmm, this doesn't include stuff written to omap.

#5 Updated by Loic Dachary almost 5 years ago

  • Regression set to No

Will you take into account stuff written to omap ? If you don't have time right now I'll change the status accordingly.

#6 Updated by Sage Weil almost 5 years ago

  • Priority changed from High to Low

#7 Updated by Sage Weil almost 3 years ago

  • Status changed from In Progress to Resolved

Also available in: Atom PDF