update comments

Author: parthea

gapic/templates/%namespace/%name_%version/%sub/services/%service/transports/rest.py.j2

@@ -259,9 +259,10 @@ class {{service.name}}RestTransport(_Base{{ service.name }}RestTransport):
             response_metadata = [(k, str(v)) for k, v in response.headers.items()]
             resp, _ = self._interceptor.post_{{ method.name|snake_case }}_with_metadata(resp, response_metadata)
             if CLIENT_LOGGING_SUPPORTED and _LOGGER.isEnabledFor(logging.DEBUG):  # pragma: NO COVER
-                {# Logging of the streaming response payload is in `google-api-core` because the response #}
-                {# type `rest_streaming.ResponseIterator` is an iterator. #}
-                {# Each response 'item' is logged at the time that it is fetched which happens in google-api-core. #}
+                {# TODO(https://github.com/googleapis/gapic-generator-python/issues/2389): #}
+                {# Depending how we want to log (a) receiving a streaming response vs (b) exposing the next streamed item to the user, we could possibly want to log something here #}
+                {# (a) should always happen in api-core #}
+                {# (b) could happen in api-core, or it could happen here when we iterate to the next streamed item that was previously received. #}
                 {% if not method.server_streaming %}
                 try:
                     response_payload = {% if method.output.ident.is_proto_plus_type %}{{ method.output.ident }}.to_json(response){% else %}json_format.MessageToJson(resp){% endif %}
@@ -271,7 +272,7 @@ class {{service.name}}RestTransport(_Base{{ service.name }}RestTransport):
                     response_payload = None
                 {% endif %}{# if not method.server_streaming #}
                 http_response = {
-                {# Not logging server streaming here. See comment above. #}
+                {# Not logging response payload for server streaming here. See comment above. #}
                 {% if not method.server_streaming %}
                 "payload": response_payload,
                 {% endif %}{# if not method.server_streaming #}

gapic/templates/%namespace/%name_%version/%sub/services/%service/transports/rest_asyncio.py.j2

@@ -220,9 +220,10 @@ class Async{{service.name}}RestTransport(_Base{{ service.name }}RestTransport):
             response_metadata = [(k, str(v)) for k, v in response.headers.items()]
             resp, _ = await self._interceptor.post_{{ method.name|snake_case }}_with_metadata(resp, response_metadata)
             if CLIENT_LOGGING_SUPPORTED and _LOGGER.isEnabledFor(logging.DEBUG):  # pragma: NO COVER
-            {# Logging of the streaming response payload is in `google-api-core` because the response #}
-            {# type `rest_streaming_async.AsyncResponseIterator` is an iterator. #}
-            {# Each response 'item' is logged at the time that it is fetched which happens in google-api-core. #}
+            {# TODO(https://github.com/googleapis/gapic-generator-python/issues/2389): #}
+            {# Depending how we want to log (a) receiving a streaming response vs (b) exposing the next streamed item to the user, we could possibly want to log something here #}
+            {# (a) should always happen in api-core #}
+            {# (b) could happen in api-core, or it could happen here when we iterate to the next streamed item that was previously received. #}
             {% if not method.server_streaming %}
                 try:
                     response_payload = {% if method.output.ident.is_proto_plus_type %}{{ method.output.ident }}.to_json(response){% else %}json_format.MessageToJson(resp){% endif %}
@@ -232,9 +233,7 @@ class Async{{service.name}}RestTransport(_Base{{ service.name }}RestTransport):
                     response_payload = None
             {% endif %}{# if not method.server_streaming #}
                 http_response = {
-                {# Logging of the streaming response payload is in `google-api-core` because the response #}
-                {# type `rest_streaming_async.AsyncResponseIterator` is an iterator. #}
-                {# Each response 'item' is logged at the time that it is fetched which happens in google-api-core. #}
+                {# Not logging response payload for server streaming here. See comment above. #}
                 {% if not method.server_streaming %}
                     "payload": response_payload,
                 {% endif %}{# if not method.server_streaming #}