kenhuuu commented on code in PR #3315:
URL: https://github.com/apache/tinkerpop/pull/3315#discussion_r2933446175
##########
gremlin-dotnet/src/Gremlin.Net/Driver/Messages/ResponseMessage.cs:
##########
@@ -21,42 +21,55 @@
#endregion
-using System;
-
namespace Gremlin.Net.Driver.Messages
{
/// <summary>
- /// The message returned from the server.
+ /// The 4.0 response message returned from the server.
/// </summary>
- /// <typeparam name="T">The type of the data returned.</typeparam>
+ /// <typeparam name="T">The type of the result data.</typeparam>
public record ResponseMessage<T>
{
/// <summary>
/// Initializes a new instance of the <see
cref="ResponseMessage{T}" /> record.
/// </summary>
- /// <param name="requestId">The identifier of the <see
cref="RequestMessage"/> that generated this <see
cref="ResponseMessage{T}"/>.</param>
- /// <param name="status">Status information about this <see
cref="ResponseMessage{T}"/>.</param>
- /// <param name="result">The result with its data and optional meta
information.</param>
- public ResponseMessage(Guid? requestId, ResponseStatus status,
ResponseResult<T> result)
+ /// <param name="bulked">Whether the response contains bulked
results.</param>
+ /// <param name="result">The deserialized result data.</param>
+ /// <param name="statusCode">The status code from the status
footer.</param>
Review Comment:
Is the term footer actually correct here? As in, are these actually
populated from the trailling headers?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
