On 07/19/2016 11:01 AM, Atin Mukherjee wrote:
On Tue, Jul 19, 2016 at 7:29 PM, Rajesh Joseph <rjoseph@xxxxxxxxxx <mailto:rjoseph@xxxxxxxxxx>> wrote: On Tue, Jul 19, 2016 at 11:23 AM, <sriram@xxxxxxxxxxxxx <mailto:sriram@xxxxxxxxxxxxx>> wrote: __ Hi Rajesh, I'd thought about moving the zfs specific implementation to something like xlators/mgmt/glusterd/src/plugins/zfs-specifs-stuffs for the inital go. Could you let me know if this works or in sync with what you'd thought about? Sriram Hi Sriram, Sorry, I was not able to send much time on this. I would prefer you move the code to xlators/mgmt/glusterd/plugins/src/zfs-specifs-stuffs How about having it under xlators/mgmt/glusterd/plugins/snapshot/src/zfs-specifs-stuffs such that in future if we have to write plugins for other features they can be segregated?
It would be nicer to avoid "specific-stuff" or similar from the naming. We can probably leave it at xlators/mgmt/glusterd/plugins/snapshot/src/zfs. The naming would be sufficient to indicate that code is specific to zfs snapshots.
-Vijay _______________________________________________ Gluster-devel mailing list Gluster-devel@xxxxxxxxxxx http://www.gluster.org/mailman/listinfo/gluster-devel
