Bug 1274990

Summary:	Enable lookup-optimize for tiered volumes
Product:	[Community] GlusterFS	Reporter:	Dan Lambright <dlambrig>
Component:	tiering	Assignee:	Dan Lambright <dlambrig>
Status:	CLOSED NOTABUG	QA Contact:	bugs <bugs>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	mainline	CC:	bugs
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2015-10-25 14:23:10 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

Description Dan Lambright 2015-10-24 15:38:44 UTC

The lookup-optimize is an important DHT optimization. Rather than sending lookups to all sub volumes, it only sends lookups to hashed sub volumes (unless there has been a configuration change). Tiered volumes can benefit from the feature in lower level DHT translators.

Comment 1 Vijay Bellur 2015-10-24 15:54:40 UTC

REVIEW: http://review.gluster.org/12421 (cluster/tier: WIP support lookup-optimize for tiered volumes) posted (#1) for review on master by Dan Lambright (dlambrig)

Comment 2 Dan Lambright 2015-10-25 14:23:10 UTC

I don't think it is needed based on experiments and code inspection: I think this is because it is only included if the voltype is "cluster/distribute" per glusterd-volume-set whereas tier uses cluster/tier. I created distributed volume with lookup-unhashed on, then attached tier. Also set lookup-unhashed to an existing volume, turned the volume off then back on. In all cases the dict reaching tier_init() did not include lookup-unhashed.

Setting to CLOSED and will reopen if necessary.