fix: Updated logging in the client pool and client factory to log information about the required transport and actual transport used. (#1853)

Author: MarkDuckworth

dev/src/index.ts

@@ -617,7 +617,12 @@ export class Firestore implements firestore.Firestore {
           );
         }
 
-        logger('Firestore', null, 'Initialized Firestore GAPIC Client');
+        logger(
+          'clientFactory',
+          null,
+          'Initialized Firestore GAPIC Client (useFallback: %s)',
+          useFallback
+        );
         return client;
       },
       /* clientDestructor= */ client => client.close()

dev/src/pool.ts

@@ -122,7 +122,12 @@ export class ClientPool<T> {
         this.concurrentOperationLimit - selectedClientRequestCount
       );
     } else {
-      logger('ClientPool.acquire', requestTag, 'Creating a new client');
+      logger(
+        'ClientPool.acquire',
+        requestTag,
+        'Creating a new client (requiresGrpc: %s)',
+        requiresGrpc
+      );
       selectedClient = this.clientFactory(requiresGrpc);
       selectedClientRequestCount = 0;
       assert(