All of lore.kernel.org
 help / color / mirror / Atom feed
* BlueStore Transaction + collection
@ 2016-07-15  0:19 Somnath Roy
  2016-07-15 13:26 ` Igor Fedotov
  0 siblings, 1 reply; 2+ messages in thread
From: Somnath Roy @ 2016-07-15  0:19 UTC (permalink / raw)
  To: Sage Weil (sweil@redhat.com); +Cc: ceph-devel (ceph-devel@vger.kernel.org)

Hi Sage/Sam,
I am seeing we are using vector of collections within _txc_add_transaction() (https://github.com/ceph/ceph/blob/master/src/os/bluestore/BlueStore.cc#L5206). I always thought all ops within a transaction will be having same collection id.
What is the scenario (or use case) where we may have ops with different collection ids within a transaction.
Also, I saw in BlueStore we are storing the first_collection (https://github.com/ceph/ceph/blob/master/src/os/bluestore/BlueStore.h#L893)  among these op->cid.
I couldn't understand the reason as only place we are using that to trim the collection cache during _txc_finish -> _osr_reap_done(). If there is multiple collections within a txc op , why only first one is used.

Thanks & Regards
Somnath
PLEASE NOTE: The information contained in this electronic mail message is intended only for the use of the designated recipient(s) named above. If the reader of this message is not the intended recipient, you are hereby notified that you have received this message in error and that any review, dissemination, distribution, or copying of this message is strictly prohibited. If you have received this communication in error, please notify the sender by telephone or e-mail (as shown above) immediately and destroy any and all copies of this message in your possession (whether hard copies or electronically stored copies).

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: BlueStore Transaction + collection
  2016-07-15  0:19 BlueStore Transaction + collection Somnath Roy
@ 2016-07-15 13:26 ` Igor Fedotov
  0 siblings, 0 replies; 2+ messages in thread
From: Igor Fedotov @ 2016-07-15 13:26 UTC (permalink / raw)
  To: Somnath Roy, Sage Weil (sweil@redhat.com)
  Cc: ceph-devel (ceph-devel@vger.kernel.org)

Somnath,

I suppose there is mandatory 'primary' collection and there might exist 
several 'auxiliary' ones within the transaction. Hence all operations 
act on the same primary collection and can optionally reference 
additional collections(s), e.g. split_collection op. Two independent 
operations on different transactions aren't allowed.
Disclaimer: Everything is IMHO based on brief code analysis.

Igor

On 15.07.2016 3:19, Somnath Roy wrote:
> Hi Sage/Sam,
> I am seeing we are using vector of collections within _txc_add_transaction() (https://github.com/ceph/ceph/blob/master/src/os/bluestore/BlueStore.cc#L5206). I always thought all ops within a transaction will be having same collection id.
> What is the scenario (or use case) where we may have ops with different collection ids within a transaction.
> Also, I saw in BlueStore we are storing the first_collection (https://github.com/ceph/ceph/blob/master/src/os/bluestore/BlueStore.h#L893)  among these op->cid.
> I couldn't understand the reason as only place we are using that to trim the collection cache during _txc_finish -> _osr_reap_done(). If there is multiple collections within a txc op , why only first one is used.
>
> Thanks & Regards
> Somnath
> PLEASE NOTE: The information contained in this electronic mail message is intended only for the use of the designated recipient(s) named above. If the reader of this message is not the intended recipient, you are hereby notified that you have received this message in error and that any review, dissemination, distribution, or copying of this message is strictly prohibited. If you have received this communication in error, please notify the sender by telephone or e-mail (as shown above) immediately and destroy any and all copies of this message in your possession (whether hard copies or electronically stored copies).
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-07-15 13:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-07-15  0:19 BlueStore Transaction + collection Somnath Roy
2016-07-15 13:26 ` Igor Fedotov

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.