On Fri, 25 Feb 2011, Yehuda Sadeh Weinraub wrote: > On Fri, Feb 25, 2011 at 10:31 AM, Colin Patrick McCabe > <colin.mccabe@xxxxxxxxxxxxx> wrote: > > I kind of want to prefix all the object operations with "obj_" just to > > make things clear. Then it would be rados_obj_rollback() / > > PoolHandle::obj_rollback(). > > What do you think? > > > > The 'o' in rados stands for object. We're definitely moving into bikeshedding territory now, but one thing Yehuda suggested: The pool handle (rados_pool_t, PoolHandle) has state associated with it, including the snap to read from or snap context to write with. That's not obvious from the name. Maybe it should be renamed rados_ioctx_t to be more clear. Then the pool we're reading to/writing from becomes one more piece of state associated with the handle. sage -- 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