Cole-Greer commented on PR #3285: URL: https://github.com/apache/tinkerpop/pull/3285#issuecomment-3629588426
+1 to Andrea's question above. Unless there are other justifications I'm not aware of, I would be more in favour of updating `makeBytecodeRequest()`, `makeStringRequest()`, `serializeMessage()`, and `deserializeMessage()` publicly accessible of wrapping them here. If we think that those methods don't provide a convenient interface for external usage, I would prefer to add dedicated public methods to `graphBinary.go` and `request.go` which are more usable. -- 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]
