Review comments.

Author: kannanjgithub

rls/src/main/java/io/grpc/rls/CachingRlsLbClient.java

@@ -307,7 +307,7 @@ private CachedRouteLookupResponse asyncRlsCall(
     }
     final SettableFuture<RouteLookupResponse> response = SettableFuture.create();
     io.grpc.lookup.v1.RouteLookupRequest routeLookupRequest = REQUEST_CONVERTER.convert(
-        requestFactory.create(routeLookupRequestKey, routeLookupReason));
+        RouteLookupRequest.create(routeLookupRequestKey.keyMap(), routeLookupReason));
     logger.log(ChannelLogLevel.DEBUG,
         "[RLS Entry {0}] Starting RouteLookup: {1}", routeLookupRequestKey, routeLookupRequest);
     rlsStub.withDeadlineAfter(callTimeoutNanos, TimeUnit.NANOSECONDS)

rls/src/main/java/io/grpc/rls/RlsProtoData.java

@@ -45,14 +45,14 @@ static RouteLookupRequestKey create(ImmutableMap<String, String> keyMap) {
   @Immutable
   abstract static class RouteLookupRequest {
 
-    // Names should match those in grpc.lookup.v1.RouteLookupRequest.Reason
+    /** Names should match those in {@link io.grpc.lookup.v1.RouteLookupRequest.Reason} */
     enum Reason {
       REASON_UNKNOWN,  // Unused
       REASON_MISS,     // No data available in local cache
       REASON_STALE;    // Data in local cache is stale
     }
 
-    // Reason for making this request.
+    /** Reason for making this request. */
     abstract Reason reason();
 
     /** Returns a map of key values extracted via key builders for the gRPC or HTTP request. */

rls/src/main/java/io/grpc/rls/RlsRequestFactory.java

@@ -93,13 +93,6 @@ RouteLookupRequestKey create(String service, String method, Metadata metadata) {
     return RouteLookupRequestKey.create(rlsRequestHeaders.buildOrThrow());
   }
 
-  /** Creates a {@link RouteLookupRequest} using the given request lookup key and reason. */
-  @CheckReturnValue
-  RouteLookupRequest create(RouteLookupRequestKey routeLookupRequestKey,
-      RouteLookupRequest.Reason reason) {
-    return RouteLookupRequest.create(routeLookupRequestKey.keyMap(), reason);
-  }
-
   private ImmutableMap.Builder<String, String> createRequestHeaders(
       Metadata metadata, List<NameMatcher> keyBuilder) {
     ImmutableMap.Builder<String, String> rlsRequestHeaders = ImmutableMap.builder();