Github user spmallette commented on a diff in the pull request:
https://github.com/apache/tinkerpop/pull/899#discussion_r206486615
--- Diff:
gremlin-server/src/main/java/org/apache/tinkerpop/gremlin/server/op/AbstractOpProcessor.java
---
@@ -252,15 +264,29 @@ protected boolean isForceFlushed(final
ChannelHandlerContext ctx, final RequestM
/**
* @deprecated As of release 3.2.2, replaced by {@link
#makeFrame(ChannelHandlerContext, RequestMessage, MessageSerializer, boolean,
List, ResponseStatusCode, Map)}.
*/
+ @Deprecated
protected static Frame makeFrame(final ChannelHandlerContext ctx,
final RequestMessage msg,
final MessageSerializer serializer,
final boolean useBinary, final List<Object> aggregate,
final ResponseStatusCode code) throws
Exception {
return makeFrame(ctx, msg, serializer, useBinary, aggregate, code,
Collections.emptyMap());
}
+ /**
+ * Caution: {@link #makeFrame(ResponseHandlerContext, RequestMessage,
MessageSerializer, boolean, List, ResponseStatusCode, Map)}
+ * should be used instead of this method whenever a {@link
ResponseHandlerContext} is available.
+ */
protected static Frame makeFrame(final ChannelHandlerContext ctx,
final RequestMessage msg,
--- End diff --
I would just deprecate this I think...I mean, if proper operation requires
that people call the new method then maybe we should push people to that method
more directly.
---
