[Bugs] [Bug 1203163] New: quota: code freeze review comments from patch# 9769
bugzilla at redhat.com
bugzilla at redhat.com
Wed Mar 18 09:56:20 UTC 2015
https://bugzilla.redhat.com/show_bug.cgi?id=1203163
Bug ID: 1203163
Summary: quota: code freeze review comments from patch# 9769
Product: GlusterFS
Version: mainline
Component: quota
Assignee: bugs at gluster.org
Reporter: vmallika at redhat.com
CC: bugs at gluster.org, gluster-bugs at redhat.com
Below review comments nned to be addressed by 3.7 code freeze:
Review comments from patch:
http://review.gluster.org/#/c/9769/24/xlators/cluster/dht/src/dht-common.c
File: xlators/cluster/dht/src/dht-common.c
Function: dht_aggregate_quota_xattr
Kaushal M 3:18 PM
>From what I understand, dht_aggregate is called only from the 2nd brick
onwards, and adds/aggregates information to the 1st bricks information. (I
might be wrong about this though).
In that case, there is no actual need to aggregate the dir_count here and
correct it later on. The correction later is just dividing the aggregated
dir_count with subvol count, which will be the same as the dir_count of 1st
brick (as directories are replicated on all DHT subvolumes).
If there was any particular reason you did it this way, can you explain it?
In case there is a real need for the correction, why not just start the count
from 2 here.
On line 350, you have a comment explaining why you are using (brick_count + 1)
as the divisor. You wouldn't need that if you started the count from 2 here.
OTOH, if you did start the count from 2 here, you'd need a comment explaining
it here.
--------------------------
Atin:
Please include remove-object functionality in inode-quota.t before code freeze.
--
You are receiving this mail because:
You are on the CC list for the bug.
You are the assignee for the bug.
More information about the Bugs
mailing list