Address PR comments

grpcgcp/gcp_multiendpoint.go

@@ -45,15 +45,15 @@ type contextMEKey int
 
 var meKey contextMEKey
 
-// NewMEContext returns a new Context that carries Multiendpoint name meName.
-func NewMEContext(ctx context.Context, meName string) context.Context {
-	return context.WithValue(ctx, meKey, meName)
+// NewMEContext returns a new Context that carries Multiendpoint name.
+func NewMEContext(ctx context.Context, name string) context.Context {
+	return context.WithValue(ctx, meKey, name)
 }
 
 // FromMEContext returns the MultiEndpoint name stored in ctx, if any.
 func FromMEContext(ctx context.Context) (string, bool) {
-	meName, ok := ctx.Value(meKey).(string)
-	return meName, ok
+	name, ok := ctx.Value(meKey).(string)
+	return name, ok
 }
 
 // GCPMultiEndpoint holds the state of MultiEndpoints-enabled gRPC client connection.
@@ -224,7 +224,7 @@ func (sm *monitoredConn) monitor() {
 		currentState = sm.conn.GetState()
 		// Inform all multiendpoints.
 		for _, me := range sm.gme.mes {
-			me.SetEndpointAvailable(sm.endpoint, currentState == connectivity.Ready)
+			me.SetEndpointAvailability(sm.endpoint, currentState == connectivity.Ready)
 		}
 	}
 }
@@ -320,7 +320,7 @@ func (gme *GCPMultiEndpoint) UpdateMultiEndpoints(meOpts *GCPMultiEndpointOption
 	for e, mc := range gme.pools {
 		s := mc.conn.GetState()
 		for _, me := range gme.mes {
-			me.SetEndpointAvailable(e, s == connectivity.Ready)
+			me.SetEndpointAvailability(e, s == connectivity.Ready)
 		}
 	}
 	return nil

grpcgcp/multiendpoint/multiendpoint.go

@@ -76,9 +76,9 @@ type MultiEndpoint interface {
 	// getting an outdated current endpoint.
 	Current() string
 
-	// SetEndpointAvailable informs MultiEndpoint when an endpoint becomes available or unavailable.
+	// SetEndpointAvailability informs MultiEndpoint when an endpoint becomes available or unavailable.
 	// This may change the current endpoint.
-	SetEndpointAvailable(e string, avail bool)
+	SetEndpointAvailability(e string, avail bool)
 
 	// SetEndpoints updates a list of endpoints:
 	//   - remove obsolete endpoints
@@ -244,16 +244,16 @@ func (me *multiEndpoint) switchFromTo(f, t *endpoint) {
 	})
 }
 
-// SetEndpointAvailable updates the state of an endpoint.
-func (me *multiEndpoint) SetEndpointAvailable(e string, avail bool) {
+// SetEndpointAvailability updates the state of an endpoint.
+func (me *multiEndpoint) SetEndpointAvailability(e string, avail bool) {
 	me.Lock()
 	defer me.Unlock()
-	me.setEndpointAvailable(e, avail)
+	me.setEndpointAvailability(e, avail)
 	me.maybeUpdateCurrent()
 }
 
 // Must be run under me.Lock.
-func (me *multiEndpoint) setEndpointAvailable(e string, avail bool) {
+func (me *multiEndpoint) setEndpointAvailability(e string, avail bool) {
 	ee, ok := me.endpoints[e]
 	if !ok {
 		return