bug of bucket instance log list?

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Yehuda,

I think there are logic error about max_entries
betweenRGWOp_BILog_List::execute() and  store->list_bi_log_entries.

Look at the codes:
in RGWOp_BILog_List::execute()
  do {

    list<rgw_bi_log_entry> entries;
    int ret = store->list_bi_log_entries(bucket_info.bucket,
                                          marker, max_entries - count,
                                          entries, &truncated);
    if (ret < 0) {
      dout(5) << "ERROR: list_bi_log_entries()" << dendl;
      return;
    }

    count += entries.size();

    send_response(entries, marker);
  } while (truncated && count < max_entries);

and RGWRados::list_bi_log_entries
int ret = cls_rgw_bi_log_list(index_ctx, oid, marker, max -
result.size(), entries, truncated);

I think there is a conflict between  "max_entries - count" and "max -
result.size()".   I think this bug is hard to test since I don't know
how the bi log creates.


Vincent
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux