Address review comments

Author: arjan-bal

examples/features/dualstack/README.md

@@ -2,7 +2,7 @@
 
 The dualstack example uses a custom name resolver that provides both IPv4 and
 IPv6 localhost endpoints for each of 3 server instances. The client will first
-use the default name resolver and load balancers which will only connect tot he
+use the default name resolver and load balancers which will only connect to the
 first server. It will then use the custom name resolver with round robin to
 connect to each of the servers in turn. The 3 instances of the server will bind
 respectively to: both IPv4 and IPv6, IPv4 only, and IPv6 only.
@@ -13,10 +13,10 @@ Three servers are serving on the following loopback addresses:
 1.  `127.0.0.1:50050`: Listening only on the IPv4 loopback address.
 1.  `[::1]:50051`: Listening only on the IPv6 loopback address.
 
-The server response will include the include their serving port and address type
-(IPv4, IPv6 or both). So the server on "127.0.0.1:50050" will reply to the RPC
-with the following message: `Greeting:Hello request:1 from server<50052> type:
-IPv4 only)`.
+The server response will include its serving port and address type (IPv4, IPv6
+or both). So the server on "127.0.0.1:50050" will reply to the RPC with the
+following message: `Greeting:Hello request:1 from server<50052> type: IPv4
+only)`.
 
 ## Try it
 

examples/features/dualstack/client/main.go

@@ -84,8 +84,7 @@ func (*exampleResolver) Scheme() string {
 }
 
 func main() {
-	// First it sends 5 requests using the default DNS and pickfirst load
-	// balancer.
+	// First send 5 requests using the default DNS and pickfirst load balancer.
 	log.Print("**** Use default DNS resolver ****")
 	target := fmt.Sprintf("localhost:%d", port1)
 	cc, err := grpc.NewClient(target, grpc.WithTransportCredentials(insecure.NewCredentials()))
@@ -122,8 +121,7 @@ func main() {
 	// Send 10 requests using the example nameresolver and round robin load
 	// balancer. These requests are evenly distributed among the 3 servers
 	// rather than favoring the server listening on both addresses because the
-	// resolver groups the 3 servers as 3 endpoints each
-	// with 2 addresses.
+	// resolver groups the 3 servers as 3 endpoints each with 2 addresses.
 	if err := waitForDistribution(ctx, client); err != nil {
 		log.Panic(err)
 	}
@@ -139,7 +137,7 @@ func waitForDistribution(ctx context.Context, client hwpb.GreeterClient) error {
 		// Since the IPv6 address comes first in the resolver list, it will be
 		// given higher priority.
 		fmt.Sprintf("[::1]:%d", port1),
-		// Server 1 is listening only on the IPv4 loopback address.
+		// Server 2 is listening only on the IPv4 loopback address.
 		fmt.Sprintf("127.0.0.1:%d", port2),
 		// Server 3 is listening only on the IPv6 loopback address.
 		fmt.Sprintf("[::1]:%d", port3),

examples/features/dualstack/server/main.go

@@ -75,7 +75,7 @@ func main() {
 		go func() {
 			defer wg.Done()
 			if err := s.Serve(lis); err != nil {
-				log.Fatalf("failed to serve: %v", err)
+				log.Panicf("failed to serve: %v", err)
 			}
 		}()
 		log.Printf("serving on %s\n", bindAddr)