Cole-Greer commented on code in PR #3330:
URL: https://github.com/apache/tinkerpop/pull/3330#discussion_r2962808931
##########
gremlin-javascript/src/main/javascript/gremlin-javascript/lib/driver/connection.ts:
##########
@@ -130,20 +103,9 @@ export default class Connection extends EventEmitter {
* @type {String}
*/
this.mimeType = options.mimeType || defaultMimeType;
Review Comment:
I don't think it's necessary, I've generally found the way the JS GLV
handles serializer and mime type configuration to be a bit of a mess. My goal
for this PR was to leave the previous WS connection arguments as in tact as
possible, with the intention to revisit all of these later to align with other
GLVs.
--
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]
