vkagamlyk commented on PR #2679: URL: https://github.com/apache/tinkerpop/pull/2679#issuecomment-2318291519
> Hey @vkagamlyk, there is a recent [dev-list post](https://lists.apache.org/thread/q7h5yzd2r064lv82njbmt6lmty4s24y7) which among other things proposes removing both traverser and bulkset from the serializers, and instead adds a "bulking" flag to GraphBinary's List type which could be used to retain the performance benefit of bulked results. Such an approach would make this PR obsolete. Do you have any thoughts on if we should proceed with the proposed changes and close out this PR, or if the proposed serializer changes should be amended to retain Traverser? I agree that the optional flag is the best solution that will allow to not add overhead and get a performance boost when needed -- 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]
