Project

General

Profile

Actions
Copy link

Bug #9342

closed

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

Added by jianpeng ma over 9 years ago. Updated about 7 years ago.

Status:
Resolved
Priority:
Low
Assignee:
jianpeng ma
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 (v1):
Crash signature (v2):

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.

Actions
Copy link
 #1

Updated by Loïc Dachary over 9 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 ?

Actions
Copy link
 #2

Updated by jianpeng ma over 9 years ago

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

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

Actions
Copy link
 #3

Updated by Loïc Dachary over 9 years ago

  • Status changed from New to In Progress

Here you go :-)

Actions
Copy link
 #4

Updated by Samuel Just over 9 years ago

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

Actions
Copy link
 #5

Updated by Loïc Dachary almost 9 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.

Actions
Copy link
 #6

Updated by Sage Weil almost 9 years ago

  • Priority changed from High to Low
Actions
Copy link
 #7

Updated by Sage Weil about 7 years ago

  • Status changed from In Progress to Resolved
Actions
Copy link

Also available in: Atom PDF